Skip to content

Add a GHA Windows test workflow? #50489

Description

@jsumners

When pull requests are opened/updated in this repo the following workflows are automatically run:

But there isn't a corresponding Windows workflow. This is causing a bit of headache for at least me in PR #48740. I don't have a Windows system to test on, but the PR needs to pass on Windows. The only way to get the PR to get run against a Windows system is for someone with privileges to add the needs-ci tag to the PR, and then it takes someone familiar with Jenkins, or at least the org's Jenkins setup, to find what the error is.

Given all of that, why isn't there a GitHub Actions workflow that runs the tests on Windows?

Activity

  1. richardlau commented on Oct 31, 2023

    @richardlau
    Member

    Given all of that, why isn't there a GitHub Actions workflow that runs the tests on Windows?

    Nobody has volunteered to implement one (i.e. no follow up to #32450 (comment)).

  2. jsumners commented on Oct 31, 2023

    @jsumners
    ContributorAuthor

    🙁

    Is there no one with Windows knowledge on the core team that could add it?

  3. bnoordhuis commented on Oct 31, 2023

    @bnoordhuis
    Member

    "Vested interest" is probably a better choice of words than "knowledge."

    I suspect the vast majority of regular contributors run Linux or macOS, not Windows. If you want to see it happen, send a pull request.

  4. jsumners commented on Oct 31, 2023

    @jsumners
    ContributorAuthor

    I have zero knowledge of building this thing in Windows. My only interest is in seeing validation on PRs without the overhead outlined above.

  5. H4ad commented on Nov 1, 2023

    @H4ad
    Member

    I have a Windows machine, I can help with that, I just need some direction on it, if you can add some issues/docs about how is working today, I can start from that, otherwise, I will try dig into the current GHA to see how it works.

    Also, do we have resources to run on Windows? I mean, I don't know if we use Runners from Github or if we run on our infrastructure.

  6. richardlau commented on Nov 1, 2023

    @richardlau
    Member

    @H4ad We have our own infrastructure but that's only used for the Jenkins CI. For anything running on GitHub actions we use the standard runners.

  7. jsumners commented on Nov 1, 2023

    @jsumners
    ContributorAuthor

    Thank you @H4ad.

  8. targos commented on Nov 1, 2023

    @targos
    Member
  9. richardlau commented on Nov 1, 2023

    @richardlau
    Member

    Also https://github.com/nodejs/node/blob/main/.github/workflows/build-windows.yml. Ideally replace this with a test-windows workflow.

  10. H4ad commented on Nov 2, 2023

    @H4ad
    Member

    Just for education purposes, what makes the CI run except for the on property? I ask this because I thought the coverage-windows should run by default on PRs but is not the case.

  11. targos commented on Nov 2, 2023

    @targos
    Member

    coverage-windows doesn't run because it is currently disabled: #50049

  12. bcoe commented on Jul 25, 2024

    @bcoe
    Contributor

    Edit: @targos @richardlau just noting that as of the time of this comment Coverage Windows is disabled manually again:

    Screenshot 2024-07-25 at 9 39 04 AM
  13. richardlau commented on Jul 25, 2024

    @richardlau
    Member

    @bcoe Building on Windows with the latest Visual Studio (i.e. the one in the GitHub Action runners) is broken due to a compiler bug.

    Refs: nodejs/build#3739
    Refs: #53863

  14. StefanStojanovic commented on Dec 19, 2024

    @StefanStojanovic
    Contributor

    There was another problem with VS 17.12, but it is resolved now. From what I see Windows GH action is running again now, so I'll close this issue.

  15. XANXAY2255 commented on Feb 18, 2025

    @XANXAY2255

    ເມື່ອການຮ້ອງຂໍການດຶງຖືກເປີດ / ປັບປຸງໃນ repo ນີ້, workflows ຕໍ່ໄປນີ້ແມ່ນເຮັດວຽກໂດຍອັດຕະໂນມັດ:

    ແຕ່ບໍ່ມີຂະບວນການເຮັດວຽກຂອງ Windows ທີ່ສອດຄ້ອງກັນ. ນີ້ແມ່ນເຮັດໃຫ້ເກີດການເຈັບຫົວເລັກນ້ອຍສໍາລັບຂ້າພະເຈົ້າຢ່າງຫນ້ອຍໃນ PR #48740 . ຂ້ອຍບໍ່ມີລະບົບ Windows ເພື່ອທົດສອບ, ແຕ່ PR ຕ້ອງຜ່ານ Windows. ວິທີດຽວທີ່ຈະໄດ້ຮັບ PR ເພື່ອດໍາເນີນການຕໍ່ກັບລະບົບ Windows ແມ່ນສໍາລັບຜູ້ທີ່ມີສິດທິພິເສດທີ່ຈະເພີ່ມ needs-ciແທັກໃສ່ PR, ແລະຫຼັງຈາກນັ້ນມັນຕ້ອງໃຊ້ຄົນທີ່ຄຸ້ນເຄີຍກັບ Jenkins, ຫຼືຢ່າງຫນ້ອຍການຕິດຕັ້ງ Jenkins ຂອງ org, ເພື່ອຊອກຫາຂໍ້ຜິດພາດ.

    ຍ້ອນສິ່ງນັ້ນທັງໝົດ, ເປັນຫຍັງບໍ່ມີຂະບວນການເຮັດວຽກຂອງ GitHub Actions ທີ່ດໍາເນີນການທົດສອບຢູ່ໃນ Windows?
    #50489

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

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions