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: Updates Deploy to Amazon ECS service task definition version and secrets names #421

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

RaisaBarreto
Copy link

Description

  • Atualiza a versão da task definition Deploy to Amazon ECS service pois a anterior parou de funcionar.
  • Atualiza nome das secrets usadas uma vez que as secrets migradas pro GitHub StonePayments mudaram de nome

Deploy em staging funcional com a atualização de versão da task definition:
https://github.com/stone-payments/superbowleto/actions/runs/12603665640

Your checklist for this pull request

🚨 Please review this items for a good pull request. 🍀

  1. I've read the project's Contributing Guidelines
  2. My commits are well written and follow pagarme/git-style-guide
  3. My changes are well covered by tests and logs
  4. I've updated the project docs (if needed)
  5. I fell safe about this implementation
  6. I feel comfortable with the code I wrote, and I'm not ashamed to show it to my friends

In a good pull request, everything above is true ☺️

Copy link

⚠️ Este repositório ainda não está catalogado no Backstage. ⚠️

Por favor, catalogue-o seguindo as instruções nesta documentação. [Via VPN].

💁 Qualquer problema ou dúvida, estamos no Slack, basta abrir um ticket no canal #help-foundation-platform.

Copy link

github-actions bot commented Jan 3, 2025

Dependency Review

✅ No vulnerabilities found.

Scanned Files

  • .github/workflows/superbowleto_deploy.yml

Copy link

sonarqubecloud bot commented Jan 3, 2025

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