search

yandex-cloud / terraform-provider-yandex

Terraform Yandex provider
https://www.terraform.io/docs/providers/yandex/
Mozilla Public License 2.0
211 stars 116 forks source link

Set value of labels in lower case #469

Open jtprogru opened 2 months ago

jtprogru commented 2 months ago

Greetings, please consider this PR. I encountered an unexpected behavior - values of any labels are passed in the case in which they were written. However, on the API side there is validation (regex: /[-_./@0-9a-z]*/), which prohibits the use of upper case.

If I understand your codebase correctly, the fix for this is very simple.

I'm sure this PR will be useful for everyone who uses task id from their task trackers in labels, because the most popular ones (Jira/YouTrack/Yandex Tracker) use uppercase as a key.

axozoid commented 3 weeks ago

it would be really helpful indeed. Hope the dev team could find some time to review the PR.