-
Notifications
You must be signed in to change notification settings - Fork 9
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
add --no-tail #427
add --no-tail #427
Conversation
Closes #426 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@jsonar-cpapke please add a description explaining why this change is needed. Thanks |
See the linked issue - #426 |
Failed. Please have a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Failed. Please have a look
@darya-abdollahzadeh
@jsonar-cpapke
The jobs failed due to a AWS permission error initially, I was waiting for confirmation that it had been fixed on your side. |
Job has been rerun and passed |
I still see two failures. If you approve I will approve the PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok.
I can't merge this PR because of the failing checks, but Terraform Cloud won't pass as it's not meant to run. @gabibeyo can you merge please? |
You accidentally opened it on master. Fixed it. |
Closes #426