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

ansible: correctly handle non-default Nomad local code paths. #144

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

jrasell
Copy link
Contributor

@jrasell jrasell commented Jun 12, 2024

When using the remote build functionality, the local Nomad code path was expected to be "nomad". This isn't the case for many people, and therefore this change fixes the behaviour, so we can handle all Nomad local code path names.

When using the remote build functionality, the local Nomad code
path was expected to be "nomad". This isn't the case for many
people, and therefore this change fixes the behaviour, so we can
handle all Nomad local code path names.
@jrasell jrasell self-assigned this Jun 12, 2024
Copy link
Contributor

@pkazmierczak pkazmierczak left a comment

Choose a reason for hiding this comment

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

Nice one, thanks James!

@jrasell jrasell merged commit f7653ae into main Jun 12, 2024
2 checks passed
@jrasell jrasell deleted the b-correctly-handle-non-default-nomad-local-code-path branch June 12, 2024 08:18
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