Merge remote-tracking branch 'origin/sebastian/prod-2721-think-up-a-t…

…est-strategy-for-plural-up' into sebastian/prod-2721-think-up-a-test-strategy-for-plural-up
pluralsh · Nov 27, 2024 · 65ebbd5 · 65ebbd5
2 parents 472d1da + 610af77
commit 65ebbd5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/e2e.yaml b/.github/workflows/e2e.yaml
@@ -95,10 +95,9 @@ jobs:
         with:
           method: files.uploadV2
           token: ${{ secrets.SLACK_BOT_TOKEN }}
-          # ${{ secrets.SLACK_CHANNEL_ID }}
           payload: |
             {
-              "channel": "C082X869RU1" ,
+              "channel_id": "${{ secrets.SLACK_CHANNEL_ID }}" ,
               "initial_comment": "Google e2e test has failed. Here are the generated files.",
               "file_uploads": [
                 {

diff --git a/pkg/up/deploy.go b/pkg/up/deploy.go
@@ -154,6 +154,7 @@ func (tf *terraformCmd) outputs() (map[string]Output, error) {
 
 func (tf *terraformCmd) run() (err error) {
 	for tf.retries >= 0 {
+		fmt.Println(tf.dir, tf.cmd, tf.args)
 		args := append([]string{tf.cmd}, tf.args...)
 		cmd := exec.Command("terraform", args...)
 		cmd.Dir = tf.dir