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

Pass the user specified token to consul maintenance primitive #73

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

Annih
Copy link
Contributor

@Annih Annih commented Jun 4, 2024

Otherwise we use the "globally configured" token, that may change at any point in the chef run (because of diplomat model). Let's specify the token when requested. i.e. all checks calls.

If an incorrect token is used, it may lacks permission to detect the node/service maintenance.

jeremy-clerc
jeremy-clerc previously approved these changes Jun 4, 2024
Annih added 2 commits June 4, 2024 16:53
Change-Id: Ibe2c225ebf04fd434ba75cbe1db416e0dc520f1b
Otherwise we use the "globally configured" token, that may change at
any point in the chef run (because of diplomat model).
Let's specify the token when requested. i.e. all checks calls.

If an incorrect token is used, it may lacks permission to detect the
node/service maintenance.

Change-Id: I00b6a512e487151931d9b811eaead162b4ddde57
@Annih Annih force-pushed the fix_consul_maintenance_primitive_token branch from 7237ee7 to 5214f49 Compare June 4, 2024 14:56
@Annih Annih requested a review from jeremy-clerc June 4, 2024 14:57
@Annih Annih dismissed jeremy-clerc’s stale review June 4, 2024 14:57

Change updated

@Annih Annih merged commit 7bdc0ae into master Jun 4, 2024
5 checks passed
@Annih Annih deleted the fix_consul_maintenance_primitive_token branch June 4, 2024 15:02
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