-
Notifications
You must be signed in to change notification settings - Fork 327
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
chore(*) prefix Kuma native tags with kuma.io
#910
Conversation
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
…uma_io # Conflicts: # pkg/xds/generator/testdata/outbound-proxy/03.envoy.golden.yaml # pkg/xds/generator/testdata/outbound-proxy/04.envoy.golden.yaml # pkg/xds/generator/testdata/profile-source/3-envoy-config.golden.yaml # pkg/xds/generator/testdata/profile-source/4-envoy-config.golden.yaml
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
This will require updates to the GUI and the docs.
…-- Marco
On Jul 20, 2020, at 8:56 PM, Nikolay Nikolaev ***@***.***> wrote:
@nickolaev requested review from @kumahq/kuma-maintainers on: #910 chore(*) prefix Kuma native tags with kuma.io as a code owner.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
The documentation PR is already linked in the description -> kumahq/kuma-website#240 @bloqhead let me know how we can align for updating the GUI. |
@@ -168,8 +168,8 @@ var _ = Describe("MonitoringAssignmentsGenerator", func() { | |||
"dataplane": "gateway-01", | |||
"region": "eu", | |||
"regions": ",eu,", | |||
"service": "gateway", | |||
"services": ",gateway,", | |||
"kuma_io_service": "gateway", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure this will break our Grafana dashboard. Can you check it?
@@ -130,7 +130,7 @@ resources: | |||
safeRegexMatch: | |||
googleRe2: | |||
maxProgramSize: 500 | |||
regex: .*&service=[^&]*frontend[,&].* | |||
regex: .*&kuma.io/service=[^&]*frontend[,&].* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lobkovilya those /
won't break anything in Regexes, right?
pkg/xds/generator/testdata/profile-source/3-envoy-config.golden.yaml
Outdated
Show resolved
Hide resolved
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
…uma_io Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Signed-off-by: Nikolay Nikolaev <nikolay.nikolaev@konghq.com>
Summary
Relpace:
service
->kuma.io/service
protocol
->kuma.io/protocol
zone
->kuma.io/zone
instance
->kuma.io/instance
Documentation