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 jobs are not acquired in when in a queue in a cluster #154

Merged
merged 1 commit into from
Mar 29, 2023

Conversation

triarius
Copy link
Contributor

@triarius triarius commented Mar 29, 2023

While the the controller was only pulling jobs from the kubernetes queue, it was starting the agents in the default queue because that's the default, and it did not specify one. With a cluster token, jobs can no longer be acquired across queues, so the API would give a 404 when the agent tried to acquire the job.

So in this PR, we add the queue to the BUILDKITE_AGENT_TAGS env variable that will be passed into the agent start container, which will then acquire the job from the correct queue.

@triarius triarius marked this pull request as draft March 29, 2023 04:05
@triarius triarius marked this pull request as ready for review March 29, 2023 04:50
@triarius triarius force-pushed the pdp-775-acquire-is-failing-for-clusters branch from 0ed27c7 to 41106e9 Compare March 29, 2023 05:04
@triarius triarius requested a review from a team March 29, 2023 05:38
Copy link

@tessereth tessereth left a comment

Choose a reason for hiding this comment

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

Seems sensible to me. I assume you've tested it and it works?

@triarius triarius merged commit b5811c8 into main Mar 29, 2023
@triarius triarius deleted the pdp-775-acquire-is-failing-for-clusters branch March 29, 2023 06:29
@triarius
Copy link
Contributor Author

@tessereth This agent acquired and ran a job in a cluster on Kubernetes and the controller was built from the commit that this branch points to.

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