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

[#742] Missing context for the status service fails deploy to dev #819

Conversation

placek
Copy link
Contributor

@placek placek commented Apr 22, 2024

Closes #742.

In response to the issue of failed deployments to the Dev environment due to missing context for the status-service, this pull request aims to address this by creating a status-service AWS registry. The goal is to ensure a successful deployment by providing the necessary context for the status-service. The changes are focused on integrating the status-service docker image utilization into the deploy stack, enabling a smooth deployment process to the Dev environment. Adjustments have been made in various files, such as Makefile, in scripts/govtool/Makefile, and docker-compose.yml.tpl, to incorporate the status-service registry details for optimized deployment functionality.

placek added 2 commits April 22, 2024 13:13
The user story described a situation where the deployment to the Dev
environment was failing due to missing context for the status-service.
The expected behavior was for the deploy to succeed. In response, a
status-service AWS registry was created to address this missing context,
ensuring that the deploy to Dev can now successfully utilize the
status-service.
The user story indicated that the deployment to the dev environment was
failing due to a missing context for the status-service. To address this
issue and enable a successful deployment, a status-service AWS registry
was created. However, to further enhance the deployment process, a
metadata-validation registry has been implemented. This addition ensures
that the necessary context is available for the deployment, optimizing
the functionality and reliability of the Dev environment.
@placek placek added 🐛 Bug Something isn't working 🏗 Infrastructure labels Apr 22, 2024
@placek placek self-assigned this Apr 22, 2024
@placek placek linked an issue Apr 22, 2024 that may be closed by this pull request
placek added 2 commits April 22, 2024 14:20
The user story highlighted the issue of a failed deployment to the dev
environment due to a missing context for the status-service. The
expected behavior was for the deploy to succeed. To address this,
changes were made to incorporate the status-service docker image
utilization into the deploy stack. The addition of the status-service
AWS registry now ensures that the necessary context is available for the
deployment process, moving the build process to the machine where the
deploy is performed, allowing for a successful deployment to the dev
environment.
… stack

The user story highlighted the issue of a failed deployment to the dev
environment due to a missing context for the status-service. The
expected behavior was for the deploy to succeed. In response to this,
changes have been made to integrate the metadata-validation docker image
into the deploy stack. By creating the necessary context for the
metadata-validation registry, the deployment process can now utilize
this image efficiently. The modifications include new files like
`Makefile` in `govtool/metadata-validation`, adjustments in
`scripts/govtool/Makefile` to incorporate the metadata-validation
registry, and changes in `docker-compose.yml.tpl` and `info.mk` to
include metadata-validation details for enhanced deployment
functionality.
@placek placek force-pushed the fix/742-missing-context-for-the-status-service-fails-deploy-to-dev branch from 3c75a4e to 553fa0e Compare April 22, 2024 12:21
@placek placek marked this pull request as ready for review April 22, 2024 12:22
@placek placek merged commit c6d5469 into develop Apr 22, 2024
@placek placek deleted the fix/742-missing-context-for-the-status-service-fails-deploy-to-dev branch April 22, 2024 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Something isn't working 🏗 Infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing context for the status service fails deploy to dev
2 participants