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

fix: hosted zone outputs #27

Merged
merged 2 commits into from
Oct 12, 2023
Merged

fix: hosted zone outputs #27

merged 2 commits into from
Oct 12, 2023

Conversation

patheard
Copy link
Member

Summary

Update the website module to use the correct hosted_zone outputs.

This also adds a step to the TF apply workflow that will post a message to the #ds-cds-ops channel if the workflow fails.

Related

Update the `website` module to use the correct `hosted_zone`
outputs.
@patheard patheard self-assigned this Oct 12, 2023
@patheard patheard requested review from daine and melaniebmn October 12, 2023 18:45
@patheard patheard changed the title Fix/hosted zone outputs fix: hosted zone outputs Oct 12, 2023
Copy link
Collaborator

@daine daine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@patheard patheard merged commit 50971b3 into main Oct 12, 2023
@patheard patheard deleted the fix/hosted-zone-outputs branch October 12, 2023 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants