Skip to content

[k2] remove wildcard prefix from confdata keys#1505

Merged
apolyakov merged 1 commit intomasterfrom
apolyakov/k2-correct-confdata-predefined-wildcards
Jan 21, 2026
Merged

[k2] remove wildcard prefix from confdata keys#1505
apolyakov merged 1 commit intomasterfrom
apolyakov/k2-correct-confdata-predefined-wildcards

Conversation

@apolyakov
Copy link
Contributor

This PR changes the behavior of the confdata_get_values_by_any_wildcard and confdata_get_values_by_predefined_wildcard functions. Previously, the returned array used the full confdata keys (including the wildcard). Now, the returned array uses keys with the wildcard part removed as it's done in old runtime.

@apolyakov apolyakov added this to the next milestone Jan 16, 2026
@apolyakov apolyakov self-assigned this Jan 16, 2026
@apolyakov apolyakov added bug Something isn't working k2 k2 related labels Jan 16, 2026
@apolyakov apolyakov marked this pull request as draft January 16, 2026 16:44
@apolyakov apolyakov force-pushed the apolyakov/k2-correct-confdata-predefined-wildcards branch from 50d0e00 to c52534d Compare January 17, 2026 17:09
@apolyakov apolyakov marked this pull request as ready for review January 18, 2026 13:21
@apolyakov apolyakov force-pushed the apolyakov/k2-correct-confdata-predefined-wildcards branch from c52534d to ef38504 Compare January 19, 2026 13:53
@apolyakov apolyakov force-pushed the apolyakov/k2-correct-confdata-predefined-wildcards branch from ef38504 to b6b3989 Compare January 20, 2026 15:39
Copy link
Contributor

@astrophysik astrophysik left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@PetrShumilov PetrShumilov left a comment

Choose a reason for hiding this comment

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

OK

@apolyakov apolyakov merged commit e5006dc into master Jan 21, 2026
7 checks passed
@apolyakov apolyakov deleted the apolyakov/k2-correct-confdata-predefined-wildcards branch January 21, 2026 10:23
denisichh pushed a commit that referenced this pull request Feb 3, 2026
* do not accept empty key and wildcard
* truncate keys by wildcard prefix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working k2 k2 related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants