Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

fix: restore _BODY substitution to Pub/Sub triggered builds #583

Merged
merged 2 commits into from
Aug 11, 2022

Conversation

grayside
Copy link
Collaborator

This was mostly removed over the course of various refactoring, but recent troubleshooting needs show it's value.

  • Use the entire Pub/Sub body as a substitution
  • Echo this object to the build log as the first step (before anything else can go wrong)
  • Re-order substitutions in terraform config to alphabetical order to match display order in console.

@grayside grayside self-assigned this Jul 28, 2022
@grayside grayside requested a review from a team as a code owner July 28, 2022 04:36
@github-actions github-actions bot added component: delivery Related to automation, testing, deployment of the application. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Jul 28, 2022
@github-actions github-actions bot added type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. and removed type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Aug 11, 2022
@grayside grayside merged commit de17ffc into main Aug 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: delivery Related to automation, testing, deployment of the application. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants