Skip to content

Conversation

@pracucci
Copy link
Contributor

What this PR does:
The new GitHub actions workflow "deploy" job (see #3368) is broken because of an issue with secrets/env variables name. In this PR I'm fixing it.

Which issue(s) this PR fixes:
N/A

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Signed-off-by: Marco Pracucci <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
@shovnik
Copy link
Contributor

shovnik commented Oct 22, 2020

Sorry about this mishap, the deploy skip made me not realize I missed changing it everywhere. Thanks for the quick fix.

@pstibrany pstibrany merged commit 17f7f97 into cortexproject:master Oct 22, 2020
@pracucci pracucci deleted the fix-deploy-via-github-actions branch October 22, 2020 15:17
@pstibrany
Copy link
Contributor

Sorry about this mishap, the deploy skip made me not realize I missed changing it everywhere. Thanks for the quick fix.

It's the job of the reviewers to not bugs like this slip in ;-) Fortunately it's easy to spot and fix. Thanks Marco!

@pracucci
Copy link
Contributor Author

No worries at all @shovnik ! I should have noticed it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants