-
Notifications
You must be signed in to change notification settings - Fork 230
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
Bug 1732364: Add jaeger permissions to create ES index templates #1699
Bug 1732364: Add jaeger permissions to create ES index templates #1699
Conversation
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Hi @pavolloffay. Thanks for your PR. I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Which branch should this go in? We do not specify ES image in Jaeger operator and rely on what ES operator provides by default. Therefore I think this should go inside 4.1 cc @objectiser? |
Yes we would need this change in both master and release-4.1 branches. |
@jcantrill @richm should I open a separate PR or do you just cherry-pick commit? |
I have submitted #1700 to backport to 4.1 branch |
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.
/ok-to-test
/retest |
@pavolloffay: This pull request references a valid Bugzilla bug. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@jcantrill @richm could this ber merged? |
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.
/lgtm
/test full-integ-aws |
/test full-integ-aws |
/bugzilla refresh |
@pavolloffay: This pull request references an invalid Bugzilla bug:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/bugzilla refresh |
@pavolloffay: This pull request references an invalid Bugzilla bug:
Comment In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/bugzilla refresh |
@pavolloffay: This pull request references a valid Bugzilla bug. The bug has been moved to the POST state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest |
1 similar comment
/retest |
/test full-integ-aws |
1 similar comment
/test full-integ-aws |
rsyslog flake? |
/test full-integ-aws |
2 similar comments
/test full-integ-aws |
/test full-integ-aws |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jcantrill, pavolloffay, richm The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@pavolloffay: The Bugzilla bug is in an unrecognized state (VERIFIED) and will not be moved to the MODIFIED state. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Jaeger now creates index templates at startup. Before it was creating an index with supplied mapping which could result in an index with incorrect mapping.
https://bugzilla.redhat.com/show_bug.cgi?id=1732364
Related Jaeger PR jaegertracing/jaeger#1627