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
[eks/argocd] Added ArgoCD notification configuration #851
Conversation
…aws-components into argocd-improvments * 'argocd-improvments' of github.com:cloudposse/terraform-aws-components: pre-commit fixes
@@ -1,4 +0,0 @@ | |||
output "metadata" { |
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.
why removing the outputs?
metadata
is OK to output
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.
Metadata outputs helm values - it is hard to read and can expose secrets
} | ||
|
||
provider "github" { | ||
base_url = local.github_webhook_enabled ? var.github_base_url : null |
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.
will it work when enabled = false
?
(we need to support enabled = false
)
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.
enabled
used in local.github_webhook_enabled
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.
please see comments
Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com>
Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com>
Co-authored-by: Andriy Knysh <aknysh@users.noreply.github.com>
…aws-components into argocd-improvments * 'argocd-improvments' of github.com:cloudposse/terraform-aws-components: pre-commit fixes
…aws-components into argocd-improvments * 'argocd-improvments' of github.com:cloudposse/terraform-aws-components: pre-commit fixes feat: spacelift/worker-pool can fully name and re-use iam role (#849)
what
why