Skip to content

V8 patch update workflow is broken #50497

Description

@targos

See https://github.com/nodejs/node/actions/runs/6680065519/job/18152806998

It's not clear why it fails. There's no error message in the output.

/cc @marco-ippolito

Activity

  1. added
    toolsIssues and PRs related to the tools directory.
    on Nov 1, 2023
  2. marco-ippolito commented on Nov 1, 2023

    @marco-ippolito
    Member

    Could it be a timeout?
    It's failing consistently
    cc @nodejs/security-wg

  3. targos commented on Nov 1, 2023

    @targos
    MemberAuthor

    Note it doesn't fail on my computer:

    $ ./tools/dep_updaters/update-v8-patch.sh
    Skipped because V8 is on the latest version.
    
  4. targos commented on Nov 1, 2023

    @targos
    MemberAuthor

    Yes, it's been failing consistently for some time.
    Here are the commits between the last successful run and the first failure: 718f62b...ff14b24

  5. marco-ippolito commented on Nov 1, 2023

    @marco-ippolito
    Member

    Note it doesn't fail on my computer:

    $ ./tools/dep_updaters/update-v8-patch.sh
    Skipped because V8 is on the latest version.
    

    same for me, I can create a branch and remove the pipe to temp output so we should see in the action as it runs

    ./tools/dep_updaters/update-v8-patch.sh > temp-output

  6. marco-ippolito commented on Nov 1, 2023

    @marco-ippolito
    Member

    https://github.com/nodejs/node/actions/runs/6717612045/job/18255788737
    with this change (./tools/dep_updaters/update-v8-patch.sh > temp-output), it works 🤔
    main...do-not-merge/fix-update-v8-action

  7. targos commented on Nov 1, 2023

    @targos
    MemberAuthor

    Can you try again with echo statements between each line so we have an idea about when it fails?

  8. marco-ippolito commented on Nov 1, 2023

    @marco-ippolito
    Member

    now it worked on main 🤔
    https://github.com/nodejs/node/actions/workflows/update-v8.yml
    that's 100% weird, on other branch it worked fine too
    could be git node v8 minor failing network calls?

  9. aduh95 commented on Nov 1, 2023

    @aduh95
    Contributor

    Could be related to nodejs/node-core-utils#722 😞

  10. targos commented on Nov 1, 2023

    @targos
    MemberAuthor

    Not sure. It's been failing consistently since June.

  11. targos commented on Nov 21, 2023

    @targos
    MemberAuthor
  12. marco-ippolito commented on Nov 22, 2023

    @marco-ippolito
    Member

    @targos should we add some logging to the script/action?

  13. targos commented on Nov 23, 2023

    @targos
    MemberAuthor

    Probably.

  14. marco-ippolito commented on Nov 28, 2023

    @marco-ippolito
    Member

    @nodejs/node-core-utils
    maybe related to nodejs/node-core-utils#743, it also fails for git node v8 minor
    The fails seems not to be consistent anymore but using a previous version 3.3.0 gives an output
    https://github.com/nodejs/node/actions/runs/7016406093/job/19087566216
    while using >4.0.0 no error message

  15. targos commented on Nov 28, 2023

    @targos
    MemberAuthor

    Ok, so the ncu bug is that it swallows the error message.
    The problem of the workflow is that it doesn't set Git name and email before running the update script.

  16. 4 remaining items

  17. targos commented on May 5, 2024

    @targos
    MemberAuthor
  18. reopened this on May 5, 2024
  19. aduh95 commented on May 5, 2024

    @aduh95
    Contributor
    Error: Command failed with exit code 1 (EPERM): git commit -m 'Update from 'Create or Update Request' action' --author 'Node.js GitHub Bot <github-bot@iojs.org>'
    

    Looks like the commit message is not correctly escaped (git commit -m 'Update from 'Create or Update Request' action'), related to gr2m/create-or-update-pull-request-action#280

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    toolsIssues and PRs related to the tools directory.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions