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.

Sign up for GitHub

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

Jump to bottom

[yugabyted] configure data_placement fails with uppercase placement info #16401

Closed
1 task done
FranckPachot opened this issue Mar 12, 2023 · 1 comment
Closed
1 task done

[yugabyted] configure data_placement fails with uppercase placement info #16401

FranckPachot opened this issue Mar 12, 2023 · 1 comment
Assignees
@gargsans-yb
Labels
area/ecosystem Label for all ecosystem related projects kind/bug This issue is a bug priority/medium Medium priority issue
Projects
DB Usability

Comments

@FranckPachot
Copy link
Contributor

FranckPachot commented Mar 12, 2023 •
edited by jira bot
Loading

Jira Link: DB-5810

Description

In https://github.com/yugabyte/yugabyte-db/blob/master/bin/yugabyted#L2176 the placement constraint is transformed to lowercase to be compared with the placement_locations array. However, we can set placement locations with uppercase and then it fails.

Example:

yugabyted start --cloud_location Cloud.Region.Zone
yugabyted configure data_placement --constraint_value=Cloud.Region.Zone

fails with:

FAILED: Invalid number of placement constraint specified.
For more information, check the logs in /root/var/logs

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@FranckPachot FranckPachot added kind/bug This issue is a bug area/ecosystem Label for all ecosystem related projects status/awaiting-triage Issue awaiting triage area/ybd yugabyted project related Github tickets. labels Mar 12, 2023
@yugabyte-ci yugabyte-ci added the priority/medium Medium priority issue label Mar 12, 2023
@yugabyte-ci yugabyte-ci removed area/ybd yugabyted project related Github tickets. status/awaiting-triage Issue awaiting triage labels Mar 17, 2023
gargsans-yb added a commit that referenced this issue Mar 28, 2023
@gargsans-yb
[#16401] [yugabyted] configure data_placement not working with cons…
8d47c2f 
…traint value with uppercases.

Summary:
While configuring a cluster with cloud location having uppercases in them, the `yugabyted
configure data_placement` command fails.

Test Plan: Manual Testing

Reviewers: nikhil

Reviewed By: nikhil

Subscribers: sgarg-yb

Differential Revision: https://phabricator.dev.yugabyte.com/D23877
gargsans-yb added a commit that referenced this issue Mar 29, 2023
@gargsans-yb
[Backport 2.17.3][#16401][yugabyted] configure data_placement not w…
86b829f 
…orking with constraint value with uppercases.

Summary:
While configuring a cluster with cloud location having uppercases in them, the `yugabyted
configure data_placement` command fails.

Test Plan: Manual Testing

Reviewers: nikhil

Reviewed By: nikhil

Subscribers: sgarg-yb

Differential Revision: https://phabricator.dev.yugabyte.com/D23934
@gargsans-yb
Copy link
Contributor

Code Fix Landed.

premkumr pushed a commit to premkumr/yugabyte-db that referenced this issue Apr 10, 2023
@gargsans-yb @premkumr
[yugabyte#16401] [yugabyted] configure data_placement not working w…
a42ce46 
…ith constraint value with uppercases.

Summary:
While configuring a cluster with cloud location having uppercases in them, the `yugabyted
configure data_placement` command fails.

Test Plan: Manual Testing

Reviewers: nikhil

Reviewed By: nikhil

Subscribers: sgarg-yb

Differential Revision: https://phabricator.dev.yugabyte.com/D23877
@nchandrappa nchandrappa added this to To-do in DB Usability May 26, 2023
@gargsans-yb gargsans-yb moved this from To-do to Done in DB Usability May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gargsans-yb gargsans-yb

Labels
area/ecosystem Label for all ecosystem related projects kind/bug This issue is a bug priority/medium Medium priority issue
Projects
DB Usability
Done
Milestone
No milestone
Development

No branches or pull requests
4 participants
@nchandrappa @yugabyte-ci @FranckPachot @gargsans-yb