8000 fix(dal): ensure ec2 qualification runs after create action by zacharyhamm · Pull Request #2692 · systeminit/si · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(dal): ensure ec2 qualification runs after create action #2692

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

Merged
merged 1 commit into from
Aug 24, 2023

Conversation

zacharyhamm
Copy link
Contributor

This adds the resource to the si:qualificationEc2CanRun function inputs to ensure that we run the qualification again after an EC2 instance has been created. Fixes the warning remaining on the instance about Key Pair creation.

@github-actions github-actions bot added the A-dal label Aug 24, 2023
@zacharyhamm
Copy link
Contributor Author

bors r+

si-bors-ng bot added a commit that referenced this pull request Aug 24, 2023
2692: fix(dal): ensure ec2 qualification runs after create action r=zacharyhamm a=zacharyhamm

This adds the `resource` to the `si:qualificationEc2CanRun` function inputs to ensure that we run the qualification again after an EC2 instance has been created. Fixes the warning remaining on the instance about Key Pair creation.

Co-authored-by: Zachary Hamm <zack@systeminit.com>
@zacharyhamm
Copy link
Contributor Author

bors r-

@si-bors-ng
Copy link
Contributor
si-bors-ng bot commented Aug 24, 2023

Canceled.

This adds the `resource` to the `si:qualificationEc2CanRun` function
inputs to ensure that we run the qualification again after an EC2
instance has been created. Fixes the warning remaining on the instance
about Key Pair creation.
@zacharyhamm zacharyhamm force-pushed the zack/eng-1866-ec2-qualification-depend-on-resource branch from 08973ab to 7ff0b29 Compare August 24, 2023 15:08
@zacharyhamm
Copy link
Contributor Author

bors r+

@si-bors-ng
Copy link
Contributor
si-bors-ng bot commented Aug 24, 2023

🕐 Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

@si-bors-ng
Copy link
Contributor
si-bors-ng bot commented Aug 24, 2023

Build succeeded:

@si-bors-ng si-bors-ng bot merged commit b0230c7 into main Aug 24, 2023
@si-bors-ng si-bors-ng bot deleted the zack/eng-1866-ec2-qualification-depend-on-resource branch August 24, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0