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

HOFF-1059: fix functional test failing in GitHub workflow #492

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

Rhodine-orleans-lindsay
Copy link
Contributor

@Rhodine-orleans-lindsay Rhodine-orleans-lindsay commented Jan 9, 2025

What?

Fix Hof automated functional test failing on GitHub Actions - HOFF-1059

Why?

  • Hof has automated workflows that run different jobs including tests when a branch is pushed to GitHub. The job that runs HOF tests is failing on the functional tests which is preventing the completion of checks.
  • Between December 5th and January 17th, there was a rollout for an update to the ubuntu operating system that the GitHub workflow is running on (See here - Ubuntu-latest workflows will use Ubuntu-24.04 image actions/runner-images#10636). As our workflow is set to run on 'ubuntu-latest' by default, it picked up the recently updated version but this is causing our functional test to break (See image in description for error log).

How?

Per the guidance in the link above I will rollback our ubuntu version and set it to 22.04 which will still be supported for 2 more years.

Testing?

Workflow tests now passing

Screenshots (optional)

Anything Else? (optional)

  • I have updated the node version the workflow runs from 14 to 20 as 14 is no longer supported and our services are updated to 20
  • I have updated the GitHub actions versions to the latest (version 4)
  • I have renamed the pr template so it can be generated automatically on pr creation. I also amend one of the checklist points from checking for green drone builds to passing workflow jobs as hof doesn't build on drone.

Check list

  • I have reviewed my own pull request for linting issues (e.g. adding new lines)
  • I have written tests (if relevant) - N/A
  • I have created a JIRA number for my branch
  • I have created a JIRA number for my commit
  • I have followed the chris beams method for my commit https://cbea.ms/git-commit/
    here is an example commit
  • Ensure workflow jobs are passing especially tests
  • I will squash the commits before merging

@Rhodine-orleans-lindsay Rhodine-orleans-lindsay force-pushed the HOFF-1059-fix-functional-test branch from 3f9f980 to 7fdf1c2 Compare January 9, 2025 13:00
@Rhodine-orleans-lindsay Rhodine-orleans-lindsay changed the title Hoff 1059 fix functional test HOFF-1059: fix functional test failing in GitHub workflow Jan 9, 2025
@Rhodine-orleans-lindsay Rhodine-orleans-lindsay force-pushed the HOFF-1059-fix-functional-test branch 2 times, most recently from 9da8a51 to b10f1da Compare January 9, 2025 13:35
@Rhodine-orleans-lindsay Rhodine-orleans-lindsay marked this pull request as ready for review January 9, 2025 13:38
- rollback ubuntu version from latest to 22.04 to fix failing workflow
- update node version to version 20
- update actions/checkout to version 4
- update actions/setup-node to version 4
- update yarn.lock
- amend and rename pr template so it will be generated automatically on pr creation
Copy link

@Adesh-Ramchurn Adesh-Ramchurn left a comment

Choose a reason for hiding this comment

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

Hi @Rhodine-orleans-lindsay just a small comment, there are some tests that are being skipped. Are they required?

@Rhodine-orleans-lindsay
Copy link
Contributor Author

Rhodine-orleans-lindsay commented Jan 9, 2025

Hi @Rhodine-orleans-lindsay just a small comment, there are some tests that are being skipped. Are they required?

Hi @Adesh-Ramchurn, no they are not required here. Those are auto-tags for versioning which only run on the master branch - see

startsWith(github.ref, 'refs/heads/master') &&

Copy link

@Adesh-Ramchurn Adesh-Ramchurn 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 clarifying that Rhodine. Code looks good

@Rhodine-orleans-lindsay Rhodine-orleans-lindsay merged commit 3c792c9 into master Jan 9, 2025
7 checks passed
@Rhodine-orleans-lindsay Rhodine-orleans-lindsay deleted the HOFF-1059-fix-functional-test branch January 9, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants