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

Fix 2.x issue #8021

Conversation

Hailong-am
Copy link
Collaborator

@Hailong-am Hailong-am commented Sep 5, 2024

Description

fix 2.x osd bootstrap issue for backport PR #8015

Issues Resolved

Screenshot

Testing the changes

Changelog

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

Copy link

codecov bot commented Sep 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.61%. Comparing base (6474a8a) to head (505229b).
Report is 1 commits behind head on backport/backport-7992-to-2.x.

Additional details and impacted files
@@                        Coverage Diff                        @@
##           backport/backport-7992-to-2.x    #8021      +/-   ##
=================================================================
+ Coverage                          60.57%   60.61%   +0.03%     
=================================================================
  Files                               3727     3727              
  Lines                              88295    88295              
  Branches                           13653    13653              
=================================================================
+ Hits                               53486    53517      +31     
+ Misses                             31555    31528      -27     
+ Partials                            3254     3250       -4     
Flag Coverage Δ
Linux_1 28.73% <0.00%> (?)
Linux_2 56.36% <100.00%> (?)
Linux_3 37.44% <0.00%> (?)
Linux_4 29.72% <0.00%> (?)
Windows_1 28.74% <0.00%> (ø)
Windows_2 56.32% <100.00%> (ø)
Windows_3 37.44% <0.00%> (ø)
Windows_4 29.72% <0.00%> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ruanyl ruanyl merged commit bb6a3bc into opensearch-project:backport/backport-7992-to-2.x Sep 5, 2024
44 of 57 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 5, 2024
Signed-off-by: Hailong Cui <ihailong@amazon.com>
(cherry picked from commit bb6a3bc)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
SuZhou-Joe pushed a commit that referenced this pull request Sep 5, 2024
(cherry picked from commit bb6a3bc)

Signed-off-by: Hailong Cui <ihailong@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>
SuZhou-Joe pushed a commit that referenced this pull request Sep 5, 2024
…ce (#8015)

* [Workspace] Hide first home breadcrumbs within workspace (#7992)

* hide first home breadcrumbs within workspace

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* Changeset file for PR #7992 created/updated

* don't relay on text to drop home

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* don't relay on text to drop home

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 63d0d32)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* fix 2.x issue (#8021)

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@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>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 6, 2024
…ce (#8015)

* [Workspace] Hide first home breadcrumbs within workspace (#7992)

* hide first home breadcrumbs within workspace

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* Changeset file for PR #7992 created/updated

* don't relay on text to drop home

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* don't relay on text to drop home

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 63d0d32)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* fix 2.x issue (#8021)

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@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>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
(cherry picked from commit ee2656b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
SuZhou-Joe pushed a commit that referenced this pull request Sep 6, 2024
…ce (#8015) (#8050)

* [Workspace] Hide first home breadcrumbs within workspace (#7992)

* hide first home breadcrumbs within workspace



* Changeset file for PR #7992 created/updated

* don't relay on text to drop home



* don't relay on text to drop home



---------



(cherry picked from commit 63d0d32)


* fix 2.x issue (#8021)



---------






(cherry picked from commit ee2656b)

Signed-off-by: Hailong Cui <ihailong@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>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants