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

feat: add Github Actions workflow to trigger VM build ♍ #263

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

alismx
Copy link
Collaborator

@alismx alismx commented Feb 3, 2025

PULL REQUEST

Summary

Related Issue

@alismx alismx self-assigned this Feb 3, 2025
@alismx alismx requested a review from rin-skylight February 3, 2025 23:47
@austin-hall-skylight austin-hall-skylight force-pushed the main branch 2 times, most recently from 0bf14c2 to d1213cc Compare February 4, 2025 16:56
@alismx alismx added the streamline eCR Issues for the Streamline eCR team label Feb 4, 2025
run: |
repo_owner="CDCgov"
repo_name="dibbs-vm"
event_type="trigger-workflow"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Informational comment...this event_type field. I was unclear on the documentation for this, but it's my understanding that we can change this type to support multiple jobs. I think. We may want to make this more specific, along with the receiving job, if we don't distinguish between which VM to build using the service field. You may already have considered this, though, so if you have, please ignore my musings!

Copy link
Collaborator Author

@alismx alismx Feb 5, 2025

Choose a reason for hiding this comment

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

I appreciate it! I was thinking about how we may want to use this event_type in the future, but for this PR, I figured I'd leave it. I may change it in the follow-up.

@alismx alismx merged commit e7d9d2a into main Feb 5, 2025
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
streamline eCR Issues for the Streamline eCR team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants