Skip to content

[Backport 2.x] Support CI for Windows and MacOS (#1164)#1167

Merged
peternied merged 2 commits into2.xfrom
backport/backport-1164-to-2.x
Nov 1, 2022
Merged

[Backport 2.x] Support CI for Windows and MacOS (#1164)#1167
peternied merged 2 commits into2.xfrom
backport/backport-1164-to-2.x

Conversation

@peternied
Copy link
Member

@peternied peternied commented Oct 28, 2022

Description

Creates a github action to install and configure Dashboards that is reused within the workflows of this repository, its multi-platform safe.

Signed-off-by: Peter Nied petern@amazon.com
(cherry picked from commit a21c7be)

Check List

  • New functionality includes testing
  • New functionality has been documented
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peternied peternied requested a review from a team October 28, 2022 16:58
@peternied peternied mentioned this pull request Oct 28, 2022
3 tasks
@codecov-commenter
Copy link

codecov-commenter commented Oct 28, 2022

Codecov Report

Merging #1167 (acf2ccb) into 2.x (a616b0a) will decrease coverage by 1.77%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x    #1167      +/-   ##
==========================================
- Coverage   74.43%   72.66%   -1.78%     
==========================================
  Files          86       88       +2     
  Lines        1874     1928      +54     
  Branches      244      250       +6     
==========================================
+ Hits         1395     1401       +6     
- Misses        424      472      +48     
  Partials       55       55              
Impacted Files Coverage Δ
...s/configuration/panels/role-view/tenants-panel.tsx
...ty-dashboards-plugin/public/utils/logout-utils.tsx
.../apps/configuration/panels/role-edit/role-edit.tsx
...tion/panels/role-view/cluster-permission-panel.tsx
...ration/panels/role-edit/index-permission-panel.tsx
...lic/apps/configuration/utils/panel-with-header.tsx
...ration/panels/audit-logging/edit-setting-group.tsx
...s/configuration/panels/audit-logging/constants.tsx
...apps/configuration/utils/loading-spinner-utils.tsx
...s/configuration/panels/tenant-list/tenant-list.tsx
... and 164 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@peternied peternied force-pushed the backport/backport-1164-to-2.x branch from 0395d12 to acf2ccb Compare October 28, 2022 18:46
Creates a github action to install and configure Dashboards that is reused within the workflows of this repository, its multi-platform safe.

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit a21c7be)
Signed-off-by: Peter Nied <petern@amazon.com>
@peternied peternied force-pushed the backport/backport-1164-to-2.x branch from acf2ccb to b0e2d4e Compare October 31, 2022 16:18
@peternied peternied added the v2.4.0 'Issues and PRs related to version v2.4.0' label Oct 31, 2022
@peternied
Copy link
Member Author

@RyanL1997 Could you take a look at this to help get it merged for 2.4?

@peternied
Copy link
Member Author

@opensearch-project/security Can I get another review on this please?

- id: branch-switch-if-possible
continue-on-error: true # Defaults onto main if the branch switch doesn't work
if: ${{ steps.osd-version.outputs.osd-version }}
run: git checkout ${{ steps.osd-version.outputs.osd-version }} || git checkout ${{ steps.osd-version.outputs.osd-x-version }}x
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this case-sensitive?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all branch and tag names are case sensitive by design

From https://stackoverflow.com/a/57754769/533057

Sounds like it.

@peternied peternied merged commit a8278c1 into 2.x Nov 1, 2022
@peternied peternied added the backport 2.4 backport to 2.4 branch label Nov 1, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 1, 2022
* Support CI for Windows and MacOS (#1164)

Creates a github action to install and configure Dashboards that is reused within the workflows of this repository, its multi-platform safe.

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit a21c7be)

* Support checkout out #.X branch types

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit a8278c1)
peternied added a commit that referenced this pull request Nov 2, 2022
* Support CI for Windows and MacOS (#1164)

Creates a github action to install and configure Dashboards that is reused within the workflows of this repository, its multi-platform safe.

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit a21c7be)

* Support checkout out #.X branch types

Signed-off-by: Peter Nied <petern@amazon.com>
(cherry picked from commit a8278c1)

Co-authored-by: Peter Nied <petern@amazon.com>
@peternied peternied deleted the backport/backport-1164-to-2.x branch November 30, 2022 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 2.4 backport to 2.4 branch v2.4.0 'Issues and PRs related to version v2.4.0'

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants