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

Update check-parameters automation to fail on AWS errors #97714

Open
6 tasks
rmtolmach opened this issue Nov 22, 2024 · 0 comments
Open
6 tasks

Update check-parameters automation to fail on AWS errors #97714

rmtolmach opened this issue Nov 22, 2024 · 0 comments
Assignees
Labels
bug Something isn't working platform-product-team

Comments

@rmtolmach
Copy link
Contributor

partially written by AI

User Story

As a PR reviewer, I want the check-parameters automation to fail when it encounters AWS permission errors, so that invalid parameter validation results are not incorrectly posted to PRs.

Issue Description

The check-parameters automation in the vsp-infra-application-manifests repo validates AWS parameters in a PR by checking their existence in Parameter Store. In https://github.com/department-of-veterans-affairs/vsp-infra-application-manifests/pull/3276, the automation encountered a permission error while attempting to check the /cms/ parameter but still posted a comment to the PR claiming the parameters were valid. See the Query for parameters... section of the action.

This issue highlights a flaw in the automation: it does not fail when the aws ssm get-parameters command returns an error. The automation should halt and report the failure instead of posting potentially incorrect validation results.

Tasks

  • Update the check-parameters script to capture errors from the aws ssm get-parameters command.
  • Ensure the script fails the workflow if any error is encountered while running the AWS command.
  • Test the updated script to ensure it properly fails on permission errors and does not post incorrect comments.

Acceptance Criteria

  • The check-parameters automation halts and fails the workflow if it encounters any AWS errors, such as permission issues.
  • No incorrect comments about parameter validity are posted to PRs when an error occurs.
  • Changes are tested and verified in a development or staging environment.

Validation

Assignee to add steps to this section. List the actions that need to be taken to confirm this issue is complete. Include any necessary links or context. State the expected outcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working platform-product-team
Projects
None yet
Development

No branches or pull requests

2 participants