Skip to content

Conversation

convenient
Copy link

A few tweaks here to update github action

Update actions to supported versions

actionlint .github/workflows/hypernode-deploy.yml
.github/workflows/hypernode-deploy.yml:16:15: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue [action]
   |
16 |       - uses: actions/checkout@v2
   |               ^~~~~~~~~~~~~~~~~~~
.github/workflows/hypernode-deploy.yml:17:15: the runner of "actions/cache@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue [action]
   |
17 |       - uses: actions/cache@v2
   |               ^~~~~~~~~~~~~~~~

This action was also prohibited without an upgrade https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Add a retention limit to the deploy artifact

To keep storage costs under control, by default aritfacts are stored for 90 days. A magento artifact can easilly be +500MB so these can add up quickly when deployments to staging are made.

Add run-name for variable name

This allows us to see which specific branch triggered the deployment from within the github actions UI

You can see my image example
Screenshot 2025-10-01 at 14 51 31

The top build was triggered with the following, which dynamically updates the run name based on the deployed branch.

name: 
run-name: Hypernode Deploy – ${{ github.ref_name }}

This can be very useful for spotting production deploys in a sea of staging deployments.

A few tweaks here to update github action

**Update actions to supported versions**
```
actionlint .github/workflows/hypernode-deploy.yml
.github/workflows/hypernode-deploy.yml:16:15: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue [action]
   |
16 |       - uses: actions/checkout@v2
   |               ^~~~~~~~~~~~~~~~~~~
.github/workflows/hypernode-deploy.yml:17:15: the runner of "actions/cache@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue [action]
   |
17 |       - uses: actions/cache@v2
   |               ^~~~~~~~~~~~~~~~
```

https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

**Add a retention limit to the deploy artifact**
To keep storage costs under control

**Add `run-name` for variable name

This allows us to see which specific branch triggered the deployment from within the github actions UI
Copy link
Member

@tdgroot tdgroot left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Some nice improvements altogether :).

While we're at it, could you update the actions so they use the latest versions? For example:

  • actions/checkout has v5
  • actions/cache has v4
  • actions/download-artifact has v5
  • actions/upload-artifact has v4
  • webfactory/ssh-agent has v0.9.1

@convenient
Copy link
Author

Hey @tdgroot

Just so i'm clear, do you mean actually update your actions within .github/workflows? or just the references in the .md files that I have already been editing :)

@poespas
Copy link
Collaborator

poespas commented Oct 2, 2025

I assume it's the markdown files, thank you for confirming :)

@convenient
Copy link
Author

Hey @poespas @tdgroot

Please let me know if you need anything else 👍

Copy link
Member

@tdgroot tdgroot left a comment

Choose a reason for hiding this comment

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

One thing left, other than that it's looking good! :)

DEPLOY_COMPOSER_AUTH: ${{ secrets.DEPLOY_COMPOSER_AUTH }}
- name: archive production artifacts
uses: actions/upload-artifact@v4
uses: actions/upload-artifact@v5
Copy link
Member

Choose a reason for hiding this comment

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

Latest major version of actions/upload-artifact is v4.

Copy link
Author

Choose a reason for hiding this comment

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

Fixed that typo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants