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

test west region runners and fluentbit aggregator #1731

Closed

Conversation

msarahan
Copy link
Contributor

@msarahan msarahan commented Nov 18, 2024

This is testing new infrastructure and should not be merged. Please ignore.

@ajschmidt8

@msarahan msarahan added the DO NOT MERGE Hold off on merging; see PR for details label Nov 18, 2024
@github-actions github-actions bot added the ci label Nov 21, 2024
@bdice bdice changed the base branch from branch-24.12 to branch-25.02 December 4, 2024 16:44
@bdice
Copy link
Contributor

bdice commented Dec 4, 2024

@msarahan Is this safe to close now?

@msarahan
Copy link
Contributor Author

msarahan commented Dec 4, 2024

Not quite. I'm still going to be using it to test rollout of rapidsai/shared-actions#28, which is a further refinement of the "dispatch actions" idea and also shifts the "summarize" part to one final job, so that we don't need to worry about whether a worker for a given job other than the final one is or is not in the VPC.

@msarahan
Copy link
Contributor Author

msarahan commented Dec 6, 2024

On second thought, I think it'll be cleaner to just open a new PR.

@msarahan msarahan closed this Dec 6, 2024
@msarahan msarahan deleted the test_west_runner_and_fluentbit branch December 6, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci DO NOT MERGE Hold off on merging; see PR for details
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants