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: cleanup model response, exception logic for ami #5

Merged
merged 2 commits into from
Sep 7, 2024

Conversation

quixoticmonk
Copy link
Collaborator

Description

  • version bumped to 0.0.2
  • Updated the AMI release info logic to handle exceptions when the release notes are not present for an AMI
  • Updated model responses to clean before sent as outputs.

Tests

Screenshot 2024-09-05 at 22 01 09
Screenshot 2024-09-06 at 11 04 42

Create

Screenshot 2024-09-06 at 11 04 02

No-op

Screenshot 2024-09-06 at 11 03 50

Delete

Screenshot 2024-09-06 at 11 03 36

@quixoticmonk quixoticmonk requested review from wellsiau-aws and gautambaghel and removed request for wellsiau-aws September 6, 2024 19:19
@wellsiau-aws
Copy link
Collaborator

/do-e2e-tests

Copy link

End to end test has been scheduled

Copy link

E2E tests in progress

Copy link

@aws-ia-automator-prod aws-ia-automator-prod bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E2E tests completed successfully

@wellsiau-aws wellsiau-aws merged commit cde1f25 into main Sep 7, 2024
3 checks passed
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