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] Fix integration labeling to identify S3 integrations #1165

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 562c3e0 from #1157.

* Switch from toast to callout for set up failures

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Fix label selection for truncated labels

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Fix button color

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Fix tests

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Remove loading progress bar

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Remove unused imports

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Refactor labels to make distinctions more semantically useful

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

---------

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
(cherry picked from commit 562c3e0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Oct 24, 2023

Codecov Report

Merging #1165 (ceaf557) into 2.x (54d125d) will decrease coverage by 0.01%.
Report is 2 commits behind head on 2.x.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x    #1165      +/-   ##
==========================================
- Coverage   44.24%   44.24%   -0.01%     
==========================================
  Files         329      329              
  Lines       19534    19542       +8     
  Branches     4718     4723       +5     
==========================================
+ Hits         8643     8646       +3     
- Misses      10324    10328       +4     
- Partials      567      568       +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 5 files with indirect coverage changes

@Swiddis Swiddis merged commit 5beb92c into 2.x Oct 24, 2023
10 of 11 checks passed
@github-actions github-actions bot deleted the backport/backport-1157-to-2.x branch October 24, 2023 00:53
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
…ect#1157) (opensearch-project#1165)

* Switch from toast to callout for set up failures

* Fix label selection for truncated labels

* Fix button color

* Fix tests

* Remove loading progress bar

* Remove unused imports

* Refactor labels to make distinctions more semantically useful

---------

(cherry picked from commit 562c3e0)

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
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 5beb92c)
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