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

Bump GitHub workflow actions to latest versions #3355

Merged

Conversation

deining
Copy link
Contributor

@deining deining commented Aug 30, 2023

What this PR does / Why we need it:

This PR bumps GitHub workflow actions to their latest versions. This avoids warning messages as seen here.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: e1775258-3325-4938-9525-9c0b2cef5854

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3355/head:pr_3355 && git checkout pr_3355
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.35.0-dev-71de4b9-amd64

@aimuz
Copy link
Collaborator

aimuz commented Aug 31, 2023

Looks good.

/lgtm

@markmandel
Copy link
Member

@Kalaiselvi84 you've worked the most on github actions, just wanted to triple check with you before approving. You see any issues?

@Kalaiselvi84
Copy link
Contributor

I will test in my fork and update here shortly..

@Kalaiselvi84
Copy link
Contributor

Kalaiselvi84 commented Sep 6, 2023

@markmandel, I tested the Label-PR script with the elevated version and it's not working as expected. Here's the link

@markmandel
Copy link
Member

@markmandel, I tested the Label-PR script with the elevated version and it's not working as expected. Here's the link

Sorry, can you point out the issue, I'm having a hard time seeing it.

@Kalaiselvi84
Copy link
Contributor

Kalaiselvi84 commented Sep 6, 2023

I received the following error:
Error retrieving files: TypeError: Cannot read properties of undefined (reading 'addLabels')

Error log: https://gist.github.com/Kalaiselvi84/bdc04f75d76008fc17ded1422cdd88f1

If I understand correctly, I've tried using label-pr with v6 and found that some of the attributes from the previous versions aren't supported by this new version.

May I know how are we testing the other Action workflows(close, stale, and obsolete)?

@markmandel
Copy link
Member

Interesting, is this because there was no label specified in the description?

image

(Although I look at the logs, it seems like there is?)

If we can't upgrade, then we can't upgrade -- warnings are not the end of the world. or flip side @deining if you are keen to update the script so it will work, that would be good too.

May I know how are we testing the other Action workflows(close, stale, and obsolete)?

Honestly, probably just making the change and seeing what happens (which is less than ideal, so we may not want to change that one).

@Kalaiselvi84
Copy link
Contributor

Kalaiselvi84 commented Sep 6, 2023

(Although I look at the logs, it seems like there is?)

Yes, I chose /kind cleanup and didn't add a new line. However, it doesn't matter.

https://screenshot.googleplex.com/5sRr2AwNauSM2w7

which is less than ideal, so we may not want to change that one

I agree with your suggestion👍🏻

@markmandel
Copy link
Member

markmandel commented Sep 6, 2023

Either way if gihub.issues.addLabels doesn't exist, it's a moot point. It's not going to work anyway.

@aimuz
Copy link
Collaborator

aimuz commented Sep 7, 2023

https://github.com/actions/github-script#apply-a-label-to-an-issue

It looks like the github-script has been tweaked a bit and should be reformatted as follows

github.rest.issues.addLabels

on:
  issues:
    types: [opened]

jobs:
  apply-label:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/github-script@v6
        with:
          script: |
            github.rest.issues.addLabels({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              labels: ['Triage']
            })

@Kalaiselvi84
Copy link
Contributor

@aimuz, Thanks for sharing github.rest.issues.addLabels. It works!
Log - https://gist.github.com/Kalaiselvi84/e65b7e9f13986565a1d6bff1b2878362

@markmandel
Copy link
Member

@deining are you able to make those updates to this PR (and also resolve the conflicts please)?

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: c2cf2482-c018-498a-ba36-3c82175ce50a

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@Kalaiselvi84
Copy link
Contributor

@deining, would you mind appending a line to the end of the obsolete, stale, and close.yaml files to address the linting issue, please?

@google-oss-prow google-oss-prow bot added size/XS and removed size/S labels Sep 8, 2023
@deining
Copy link
Contributor Author

deining commented Sep 8, 2023

@deining, would you mind appending a line to the end of the obsolete, stale, and close.yaml files to address the linting issue, please?

Done.

I also rebase my branch, it is not out-of-date any more.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 41bf4813-19c4-4824-bcc8-33d6cc490bec

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 08a3d078-275e-4fbd-b7a6-46a4e13c1bbc

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@google-oss-prow google-oss-prow bot added size/S and removed size/XS labels Sep 8, 2023
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: e68928c8-fd48-469e-9d87-6bd717a411d7

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/3355/head:pr_3355 && git checkout pr_3355
  • helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.35.0-dev-2f0b020-amd64

Copy link
Collaborator

@aimuz aimuz left a comment

Choose a reason for hiding this comment

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

/lgtm

@google-oss-prow google-oss-prow bot added the lgtm label Sep 8, 2023
@markmandel
Copy link
Member

@Kalaiselvi84 you have the most experience with these tools - any objection to approving and merging?

@Kalaiselvi84
Copy link
Contributor

LGTM!

@Kalaiselvi84 Kalaiselvi84 self-assigned this Sep 8, 2023
@Kalaiselvi84 Kalaiselvi84 self-requested a review September 8, 2023 22:42
@Kalaiselvi84 Kalaiselvi84 removed their assignment Sep 8, 2023
@Kalaiselvi84 Kalaiselvi84 removed their request for review September 8, 2023 22:43
@markmandel markmandel merged commit d65a506 into googleforgames:main Sep 8, 2023
3 checks passed
@markmandel
Copy link
Member

HERE WE GO 😃

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: aimuz, deining, markmandel

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@deining deining deleted the bump-github-workflow-actions branch September 10, 2023 17:46
@Kalaiselvi84 Kalaiselvi84 added kind/cleanup Refactoring code, fixing up documentation, etc and removed kind/other labels Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved kind/cleanup Refactoring code, fixing up documentation, etc lgtm size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants