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

Fix warnings & deprecation messages in GitHub actions #1

Closed
wants to merge 6 commits into from

Conversation

ljubon
Copy link
Collaborator

@ljubon ljubon commented Mar 8, 2024

What type of PR is this?

CI: Resolve deprecation & warning messages in workflows

What this PR does / why we need it:

Node16 has been out of support since September 2023. - more info in the post

  • Add GitHub actions for DependaBot (commit)
  • Bump the steps where deprecation/warnings are shown in the job (commit)

Optional

  • 🛠️ Proposed action: Open a PR which bump the version in this step which is used in Slack CI Alerts workflow and currently doesn't address this issue

Which issue(s) this PR fixes:

https://github.com/G-Research/gr-oss/issues/586


Testing results

As the desired state is a job without warnings/deprecation messages, in this commit I removed some of the conditions in Armada workflow in order to test all workflows and their actions

FAILED (need action update) 🟠

Slack CI Alerts

PASSED 🟢

CI / Build / Tests / Lint

Python Client

Python Airflow Operator

CodeQL

Release Armada components
(workflow failed due to missing tag, but no deprecated/warnings are shown)

Release Armada components - RC
(workflow failed due to missing tag, but no deprecated/warnings are shown)

Check required jobs
(workflow failed due to missing ref, but no github/scripts@v7 showed there is no warnings/deprecatations)

@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch 2 times, most recently from 2fa0107 to 5ac6aa2 Compare March 13, 2024 14:25
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from 5ac6aa2 to d2d7a66 Compare March 13, 2024 14:35
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 14:35 — with GitHub Actions Failure
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from d2d7a66 to ba8b134 Compare March 13, 2024 14:43
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 14:43 — with GitHub Actions Failure
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 14:43 — with GitHub Actions Failure
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 15:06 — with GitHub Actions Failure
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from ba8b134 to e0a51e6 Compare March 13, 2024 15:08
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 15:08 — with GitHub Actions Failure
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 15:08 — with GitHub Actions Failure
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from e0a51e6 to b48f43d Compare March 13, 2024 15:11
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from b48f43d to 6b6f948 Compare March 13, 2024 15:12
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from 6b6f948 to 3b7c77b Compare March 13, 2024 15:17
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 15:17 — with GitHub Actions Failure
@ljubon ljubon had a problem deploying to armada-dockerhub March 13, 2024 15:17 — with GitHub Actions Failure
@ljubon ljubon changed the title - Enable GitHub actions scanning for updates with dependabot Fix warnings & deprecation messages in GitHub actions Mar 13, 2024
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from 3b7c77b to 9bfb211 Compare March 13, 2024 15:41
@pavlovic-ivan
Copy link

I have one general comment. As in other PRs, let's focus only on what the scope of the PR is, so in this case, no need for formatting changes, just updating the versions. Also, please provide working examples, with updated actions versions.

@ljubon ljubon requested a review from pavlovic-ivan March 15, 2024 10:07
@ljubon
Copy link
Collaborator Author

ljubon commented Mar 15, 2024

I have one general comment. As in other PRs, let's focus only on what the scope of the PR is, so in this case, no need for formatting changes, just updating the versions. Also, please provide working examples, with updated actions versions.

Thanks Ivan 🙏
I reverted changes which are not part of the scope, but was required to provide example workflows and results

Copy link

@pavlovic-ivan pavlovic-ivan left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @ljubon! 🚀

@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch 3 times, most recently from 88ce995 to 5e9fe9c Compare March 15, 2024 12:17
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch 4 times, most recently from 7df5b71 to bec1cb5 Compare March 28, 2024 17:49
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from bec1cb5 to 9cabd29 Compare April 3, 2024 10:49
ljubon added 6 commits April 22, 2024 15:30
Signed-off-by: Ljubo Nikolic <[email protected]>

Signed-off-by: Ljubo Nikolić <[email protected]>
Signed-off-by: Ljubo Nikolic <[email protected]>

Signed-off-by: Ljubo Nikolić <[email protected]>
Signed-off-by: Ljubo Nikolic <[email protected]>

Signed-off-by: Ljubo Nikolić <[email protected]>
Signed-off-by: Ljubo Nikolic <[email protected]>

Signed-off-by: Ljubo Nikolić <[email protected]>
Signed-off-by: Ljubo Nikolic <[email protected]>

Signed-off-by: Ljubo Nikolić <[email protected]>
@ljubon ljubon force-pushed the oss-586-resolve-deprecation-and-warnings branch from 9cabd29 to b350430 Compare April 22, 2024 13:30
@ljubon ljubon force-pushed the master branch 2 times, most recently from 9fabcd0 to e4e896e Compare June 20, 2024 07:49
@ljubon ljubon closed this Jul 24, 2024
@ljubon
Copy link
Collaborator Author

ljubon commented Jul 24, 2024

closing internal PR as it's merged upstream

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