Skip to content

ci: Pin GitHub Actions to full-length commit SHAs#1445

Merged
tandonks merged 5 commits into
opensearch-project:mainfrom
skumawat2025:pin-gha-to-commit-sha
Jun 25, 2026
Merged

ci: Pin GitHub Actions to full-length commit SHAs#1445
tandonks merged 5 commits into
opensearch-project:mainfrom
skumawat2025:pin-gha-to-commit-sha

Conversation

@skumawat2025

@skumawat2025 skumawat2025 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

The repository now enforces the GitHub policy requiring all Actions to be pinned to a full-length commit SHA. Workflows referencing actions by tag (e.g. actions/checkout@v3) fail at the prepare-actions step, which blocks CI on every pull request.

Pin every action - and the get-ci-image-tag reusable workflow - to its current commit SHA, keeping the previous tag as a trailing comment for readability and future updates.

Changes done in other plugins/repos: opensearch-project/dashboards-assistant#699

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • 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.

Sandeep Kumawat added 3 commits June 25, 2026 11:45
The repository now enforces the GitHub policy requiring all Actions to
be pinned to a full-length commit SHA. Workflows referencing actions by
tag (e.g. actions/checkout@v3) fail at the prepare-actions step, which
blocks CI on every pull request.

Pin every action - and the get-ci-image-tag reusable workflow - to its
current commit SHA, keeping the previous tag as a trailing comment for
readability and future updates.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
The prior commit pinned actions in .github/workflows but missed the
composite action at .github/actions/run-cypress-tests/action.yaml, which
the Cypress jobs invoke. Its actions (setup-java, checkout, setup-node,
nick-fields/retry, cypress-io/github-action, upload-artifact) are now
pinned to full commit SHAs as well, so the SHA-pinning policy no longer
blocks the Cypress workflows.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
OpenSearch Dashboards core (checked out at OPENSEARCH_DASHBOARDS_VERSION
in CI) now requires @babel/runtime ^7.29.7 via @osd/interpreter, while
this plugin declared ^7.26.10. The osd bootstrap single_version_dependencies
check fails on the mismatch, blocking unit tests.

Bump @babel/runtime to ^7.29.7 (resolutions and devDependencies) to match
core.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
@skumawat2025

Copy link
Copy Markdown
Contributor Author

Failing E2E tests are related to rollups, data_stream pages and are unrelated to the above changes.

@skumawat2025

Copy link
Copy Markdown
Contributor Author

Run binary installation check is failing due to upstream repo issues. Upstream repos needs to update commit SHA,

Download action repository 'actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744' (SHA:f43a0e5ff2bd294095638e18286ca9a3d1956744)
Download action repository 'derek-ho/start-opensearch@e9060d3df24b30cdbfac62d749fec3367dade572' (SHA:e9060d3df24b30cdbfac62d749fec3367dade572)
Download action repository 'derek-ho/setup-opensearch-dashboards@f64b331c58fe5b0a6533e76da1b46cb9d11bcaef' (SHA:f64b331c58fe5b0a6533e76da1b46cb9d11bcaef)

@tandonks tandonks left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Since failures unrelated to current changes, good to merge

cwperks and others added 2 commits June 25, 2026 15:01
…nsearch-project#1442)

Replace derek-ho/start-opensearch and derek-ho/setup-opensearch-dashboards
with opensearch-project/opensearch-build/.github/actions equivalents pinned
to SHA dbcc45dc3e390e54f1917be9c725450548416ce6.

Signed-off-by: Craig Perkins <cwperx@amazon.com>
@tandonks tandonks merged commit cb2ffd5 into opensearch-project:main Jun 25, 2026
8 of 10 checks passed
@opensearch-ci-bot

Copy link
Copy Markdown
Contributor

The backport to 2.19 failed. Please backport manually. See failed workflow run: https://github.com/opensearch-project/index-management-dashboards-plugin/actions/runs/28208244687

Hailong-am pushed a commit to Hailong-am/index-management-dashboards-plugin that referenced this pull request Jun 26, 2026
…#1445)

* ci: Pin GitHub Actions to full-length commit SHAs

The repository now enforces the GitHub policy requiring all Actions to
be pinned to a full-length commit SHA. Workflows referencing actions by
tag (e.g. actions/checkout@v3) fail at the prepare-actions step, which
blocks CI on every pull request.

Pin every action - and the get-ci-image-tag reusable workflow - to its
current commit SHA, keeping the previous tag as a trailing comment for
readability and future updates.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* ci: Pin actions in run-cypress-tests composite action to SHAs

The prior commit pinned actions in .github/workflows but missed the
composite action at .github/actions/run-cypress-tests/action.yaml, which
the Cypress jobs invoke. Its actions (setup-java, checkout, setup-node,
nick-fields/retry, cypress-io/github-action, upload-artifact) are now
pinned to full commit SHAs as well, so the SHA-pinning policy no longer
blocks the Cypress workflows.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* fix: Align @babel/runtime with OpenSearch Dashboards core

OpenSearch Dashboards core (checked out at OPENSEARCH_DASHBOARDS_VERSION
in CI) now requires @babel/runtime ^7.29.7 via @osd/interpreter, while
this plugin declared ^7.26.10. The osd bootstrap single_version_dependencies
check fails on the mismatch, blocking unit tests.

Bump @babel/runtime to ^7.29.7 (resolutions and devDependencies) to match
core.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* Update GitHub actions to use official opensearch-project actions (opensearch-project#1442)

Replace derek-ho/start-opensearch and derek-ho/setup-opensearch-dashboards
with opensearch-project/opensearch-build/.github/actions equivalents pinned
to SHA dbcc45dc3e390e54f1917be9c725450548416ce6.

Signed-off-by: Craig Perkins <cwperx@amazon.com>

---------

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Craig Perkins <cwperx@amazon.com>
Hailong-am pushed a commit to Hailong-am/index-management-dashboards-plugin that referenced this pull request Jun 26, 2026
…#1445)

* ci: Pin GitHub Actions to full-length commit SHAs

The repository now enforces the GitHub policy requiring all Actions to
be pinned to a full-length commit SHA. Workflows referencing actions by
tag (e.g. actions/checkout@v3) fail at the prepare-actions step, which
blocks CI on every pull request.

Pin every action - and the get-ci-image-tag reusable workflow - to its
current commit SHA, keeping the previous tag as a trailing comment for
readability and future updates.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* ci: Pin actions in run-cypress-tests composite action to SHAs

The prior commit pinned actions in .github/workflows but missed the
composite action at .github/actions/run-cypress-tests/action.yaml, which
the Cypress jobs invoke. Its actions (setup-java, checkout, setup-node,
nick-fields/retry, cypress-io/github-action, upload-artifact) are now
pinned to full commit SHAs as well, so the SHA-pinning policy no longer
blocks the Cypress workflows.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* fix: Align @babel/runtime with OpenSearch Dashboards core

OpenSearch Dashboards core (checked out at OPENSEARCH_DASHBOARDS_VERSION
in CI) now requires @babel/runtime ^7.29.7 via @osd/interpreter, while
this plugin declared ^7.26.10. The osd bootstrap single_version_dependencies
check fails on the mismatch, blocking unit tests.

Bump @babel/runtime to ^7.29.7 (resolutions and devDependencies) to match
core.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* Update GitHub actions to use official opensearch-project actions (opensearch-project#1442)

Replace derek-ho/start-opensearch and derek-ho/setup-opensearch-dashboards
with opensearch-project/opensearch-build/.github/actions equivalents pinned
to SHA dbcc45dc3e390e54f1917be9c725450548416ce6.

Signed-off-by: Craig Perkins <cwperx@amazon.com>

---------

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
tandonks pushed a commit that referenced this pull request Jun 26, 2026
* Bump lodash from 4.17.21 to 4.18.0 to fix CVE-2026-4800

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

* Bump picomatch to 2.3.2 to fix CVE-2026-33671

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

* Bump serialize-javascript to 7.0.3 to fix GHSA-5c6j-r48x-rmvq

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

* add yarn.lock

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

* ci: Pin GitHub Actions to full-length commit SHAs (#1445)

* ci: Pin GitHub Actions to full-length commit SHAs

The repository now enforces the GitHub policy requiring all Actions to
be pinned to a full-length commit SHA. Workflows referencing actions by
tag (e.g. actions/checkout@v3) fail at the prepare-actions step, which
blocks CI on every pull request.

Pin every action - and the get-ci-image-tag reusable workflow - to its
current commit SHA, keeping the previous tag as a trailing comment for
readability and future updates.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* ci: Pin actions in run-cypress-tests composite action to SHAs

The prior commit pinned actions in .github/workflows but missed the
composite action at .github/actions/run-cypress-tests/action.yaml, which
the Cypress jobs invoke. Its actions (setup-java, checkout, setup-node,
nick-fields/retry, cypress-io/github-action, upload-artifact) are now
pinned to full commit SHAs as well, so the SHA-pinning policy no longer
blocks the Cypress workflows.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* fix: Align @babel/runtime with OpenSearch Dashboards core

OpenSearch Dashboards core (checked out at OPENSEARCH_DASHBOARDS_VERSION
in CI) now requires @babel/runtime ^7.29.7 via @osd/interpreter, while
this plugin declared ^7.26.10. The osd bootstrap single_version_dependencies
check fails on the mismatch, blocking unit tests.

Bump @babel/runtime to ^7.29.7 (resolutions and devDependencies) to match
core.

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>

* Update GitHub actions to use official opensearch-project actions (#1442)

Replace derek-ho/start-opensearch and derek-ho/setup-opensearch-dashboards
with opensearch-project/opensearch-build/.github/actions equivalents pinned
to SHA dbcc45dc3e390e54f1917be9c725450548416ce6.

Signed-off-by: Craig Perkins <cwperx@amazon.com>

---------

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Craig Perkins <cwperx@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>

* yarn.lock

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

* add missing distribution

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

* Revert serialize-javascript 7.0.3 bump (incompatible with Node 18)

serialize-javascript 7.x requires Node >= 20 but CI runs Node 18.19.0,
causing yarn install to fail with engine incompatibility error.

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

* Remove verify-binary-installation workflow

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

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
Signed-off-by: Craig Perkins <cwperx@amazon.com>
Co-authored-by: Sandeep Kumawat <2025sandeepkumawat@gmail.com>
Co-authored-by: Sandeep Kumawat <skumwt@amazon.com>
Co-authored-by: Craig Perkins <cwperx@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.

5 participants