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

Update allocations.go #17726

Merged
merged 3 commits into from
Jun 26, 2023
Merged

Update allocations.go #17726

merged 3 commits into from
Jun 26, 2023

Conversation

sejalapeno
Copy link
Contributor

@sejalapeno sejalapeno commented Jun 26, 2023

updated missing client status "unknown" #17688

Fixes #17688

updated missing client status "unknown" hashicorp#17688
@hashicorp-cla
Copy link

hashicorp-cla commented Jun 26, 2023

CLA assistant check
All committers have signed the CLA.

@shoenig
Copy link
Member

shoenig commented Jun 26, 2023

Thanks @sejalapeno! Do you mind amending the PR with a changelog entry? You can create one just by running make cl

@sejalapeno
Copy link
Contributor Author

sejalapeno commented Jun 26, 2023

Sure, after running make cl , it is asking me type from 1 to 6

@shoenig
Copy link
Member

shoenig commented Jun 26, 2023

Ah yeah @sejalapeno it's a little wizzard that prompts a few questions, so that it can generate the properly formatted changelog entry. This is a bug fix, so pick whichever number indicates it's for a bug fix.

@shoenig shoenig added backport/1.3.x backport to 1.3.x release line backport/1.4.x backport to 1.4.x release line backport/1.5.x backport to 1.5.x release line labels Jun 26, 2023
@shoenig shoenig added this to the 1.6.0 milestone Jun 26, 2023
@sejalapeno
Copy link
Contributor Author

I hope the changes are what was required

.changelog/17726.txt Outdated Show resolved Hide resolved
adding relevant desc.

Co-authored-by: Seth Hoenig <shoenig@duck.com>
@sejalapeno
Copy link
Contributor Author

sejalapeno commented Jun 26, 2023

Is there anything from my end that is supposed to be done? Because the test ui workflow #2 attempt failed

@shoenig
Copy link
Member

shoenig commented Jun 26, 2023

The UI tests are nothing to worry about here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.3.x backport to 1.3.x release line backport/1.4.x backport to 1.4.x release line backport/1.5.x backport to 1.5.x release line
Projects
None yet
Development

Successfully merging this pull request may close these issues.

api: add missing client status "unknown"
3 participants