Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixing up CI and git hooks #2

Merged
merged 2 commits into from
Jun 7, 2018
Merged

Conversation

brandonjbjelland
Copy link
Contributor

PR o'clock

Description

Attempting CI fixes and repaired the git hooks.

Checklist

  • terraform fmt and terraform validate both work from the root and examples/alb_test_fixture directories (look in CI for an example) - done
  • Tests for the changes have been added and passing (for bug fixes/features) - none necessary
  • Test results are pasted in this PR (in lieu of CI) - trying to fix this
  • Docs have been added/updated (for bug fixes/features) - CHANGELOG updated
  • Any breaking changes are noted in the description above - None

@brandonjbjelland
Copy link
Contributor Author

brandonjbjelland commented Jun 7, 2018

CI isn't going to pass here yet (I think it's AWS provider dependent - ran into this on ALB) but I'm going to file an issue on the final CI command not running properly against this repo but also on an upstream module or two. The random_string outputs error bites me occasionally and I have no good reason to say why. The other error is along the same lines and potentially the same bug.

@brandonjbjelland brandonjbjelland merged commit fbe64df into master Jun 7, 2018
@brandonjbjelland brandonjbjelland deleted the fix/git_hooks_and_ci branch June 7, 2018 10:14
dahateb referenced this pull request in TierMobility/terraform-aws-eks Aug 22, 2019
max-rocket-internet pushed a commit that referenced this pull request Jan 27, 2020
…701)

* Configurable local exec command for waiting until cluster is healthy

* readme

* line feeds

* format

* fix readme

* fix readme

* Configurable local exec command for waiting until cluster is healthy (#1)

* Configurable local exec command for waiting until cluster is healthy

* readme

* line feeds

* format

* fix readme

* fix readme

* change log

* Configurable local exec wait 4 cluster op (#2)

* Configurable local exec command for waiting until cluster is healthy

* readme

* line feeds

* format

* fix readme

* fix readme

* change log

* changelog (#3)

* Changelog (#4)

* changelog

* changelog

* simplify wait_for_cluster command

* readme

* no op for manage auth false

* formatting

* docs? not sure

* linter

* specify dependency to wait for cluster more accurately
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant