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.

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

[Backport 2.x] Allow patch on allowedRoles #1144

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport cccfebf from #1143.

* Allow patch on allowedRoles

Signed-off-by: Derek Ho <[email protected]>

* Change for patch

Signed-off-by: Derek Ho <[email protected]>

* Address PR comments

Signed-off-by: Derek Ho <[email protected]>

---------

Signed-off-by: Derek Ho <[email protected]>
(cherry picked from commit cccfebf)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Oct 20, 2023

Codecov Report

Merging #1144 (e416e1e) into 2.x (ea85837) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x    #1144      +/-   ##
==========================================
+ Coverage   44.22%   44.24%   +0.01%     
==========================================
  Files         327      329       +2     
  Lines       19477    19532      +55     
  Branches     4706     4719      +13     
==========================================
+ Hits         8614     8641      +27     
- Misses      10297    10324      +27     
- Partials      566      567       +1     
Flag Coverage Δ
dashboards-observability 44.24% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 4 files with indirect coverage changes

@derek-ho derek-ho merged commit d5e7636 into 2.x Oct 23, 2023
11 checks passed
@derek-ho derek-ho deleted the backport/backport-1143-to-2.x branch October 23, 2023 18:28
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
…ject#1144)

* Allow patch on allowedRoles

* Change for patch

* Address PR comments

---------

(cherry picked from commit cccfebf)

Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit d5e7636)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants