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

Update GH Action Versions used in Workflows #1339

Merged
merged 4 commits into from
Feb 22, 2024

Conversation

Mythicaeda
Copy link
Contributor

@Mythicaeda Mythicaeda commented Feb 17, 2024

  • Tickets addressed: Hotfix
  • Review: By commit
  • Merge strategy: Merge (no squash)

Description

Github actions are transitioning from Node 16 to Node 20. This PR is to update our workflows to use the node 20 versions of all actions (v4).

There were some breaking changes in the upload/download artifact action, that will impact our usage, specifically being unable to upload two artifacts with the same name. This specifically impacts our Security Scan, Publish, and Test workflows.

Verification

Successful execution of each workflow with no Node16 warnings will validate this PR:

  • CLOC
  • DB Migrations
  • Load Test
  • Test/E2E Tests
  • Create JNISPICE
  • Deploy to GH Pages
  • Publish (Deploy step is untestable due to branch restrictions)
  • Security Scan

Documentation

No doc updates needed.

@Mythicaeda Mythicaeda added the ci Changes to our CI configuration files and scripts label Feb 17, 2024
@Mythicaeda Mythicaeda self-assigned this Feb 17, 2024
@Mythicaeda Mythicaeda requested a review from a team as a code owner February 17, 2024 00:58
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from 2b64918 to bc9b971 Compare February 17, 2024 00:58
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from bc9b971 to 8767ae6 Compare February 20, 2024 16:48
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from 8767ae6 to 2359490 Compare February 20, 2024 18:41
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from 2359490 to 808b47d Compare February 20, 2024 18:43
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from 808b47d to f429962 Compare February 21, 2024 17:32
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from f429962 to 6f92070 Compare February 21, 2024 17:33
@Mythicaeda Mythicaeda force-pushed the ci/upgrade-workflow-actions branch from 6f92070 to d8ab0cf Compare February 21, 2024 20:26
Copy link
Contributor

@skovati skovati left a comment

Choose a reason for hiding this comment

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

This looks great! I'll just note that we'll need to make these changes on other Aerie repos (UI and Gateway) as well.

Copy link
Collaborator

@dandelany dandelany left a comment

Choose a reason for hiding this comment

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

👍 looks good to me also - chatted with @Mythicaeda yesterday about a couple questions but I'm happy with this! For reference, this is the successful run of the Publish workflow (not run by PR Checks like the others): https://github.com/NASA-AMMOS/aerie/actions/runs/7995540299

One note: as a result of the changes to upload_artifact noted by @Mythicaeda above, the "Security Scan" workflow now creates two artifacts called "Security Scan - java" and "Security Scan - javascript". When we do a release, the "Security Scan" artifact was attached to the release by the workflows in aerie-release, so I will add a PR to aerie-release to look for the two separate scans in future instead of the one combined one.

@Mythicaeda Mythicaeda merged commit af5e597 into develop Feb 22, 2024
22 checks passed
@Mythicaeda Mythicaeda deleted the ci/upgrade-workflow-actions branch February 22, 2024 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Changes to our CI configuration files and scripts
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants