codeandpepper / janush

Set up a modern codebase for cloud-native web app with authentication features by running one command
https://janush.dev
MIT License
14 stars 1 forks source link

bugfix: janush.app deployment upgrade #211

Closed przemyslaw-rzasa-cp closed 2 years ago

przemyslaw-rzasa-cp commented 2 years ago

Problem

janush.app pipeline has been failing on (WEB) Deployment step. The reason was a fact that some time ago CfnOutput's export name was renamed (thanks @SebastianGoral for the info), but update on github actions yaml file was omitted.

Solution

Update export name. Successful pipeline

How to avoid similar situation in the future?

There has been created an issue already, which should partially cover the problem in the future. Partially, as it will rather stick only to merges into main branch (I'm afraid, that checking 2 pipelines on each PR update may be too costly). At least we will be able to monitor merge downstream pipelines in a more sensible way. For ease of debugging please always also add a meaningful comments to your PRs. Cheers!