Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

Update .e2e-tests.yaml to add windows srv 2022 #2023

Closed
wants to merge 4 commits into from

Conversation

jlind23
Copy link
Contributor

@jlind23 jlind23 commented Jan 19, 2022

Add Windows server 2022 as an operating system on which fleet server and elastic agent tests will be deployed.

@jlind23 jlind23 added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Jan 19, 2022
@jlind23 jlind23 requested a review from a team January 19, 2022 10:23
@jlind23 jlind23 self-assigned this Jan 19, 2022
@mergify
Copy link
Contributor

mergify bot commented Jan 19, 2022

This pull request does not have a backport label. Could you fix it @jlind23? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 7./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@mergify mergify bot added the backport-skip Skip notification from the automated backport with mergify label Jan 19, 2022
@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 19, 2022

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@jlind23
Copy link
Contributor Author

jlind23 commented Jan 19, 2022

@mdelapenya it seems to be the same test failing on each PR. Can we fix it?

@mdelapenya
Copy link
Contributor

@mdelapenya it seems to be the same test failing on each PR. Can we fix it?

The windows stage did fail: https://beats-ci.elastic.co/blue/organizations/jenkins/e2e-tests%2Fe2e-testing-mbp/detail/PR-2023/1/pipeline, @adam-stokes is looking into it.

And for the existing failure, there is an open issue, but I think it's not related to the tests but to the elastic-agent container. Please more details here: #1992, which links to elastic/beats#29811 as root cause

@jlind23
Copy link
Contributor Author

jlind23 commented Jan 24, 2022

@adam-stokes @mdelapenya anything I can do here on my end?

@adam-stokes
Copy link
Contributor

This requires a bit more work to support it in ansible, I'll need to get some additional tasks written to support winrm etc

@jlind23
Copy link
Contributor Author

jlind23 commented Jan 25, 2022

@adam-stokes Should I close it for now? I also wanted to add Windows 11 but i'm quite sure I am going to face the same issue right?

@mergify
Copy link
Contributor

mergify bot commented Jan 25, 2022

This pull request is now in conflict. Could you fix it @jlind23? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b jlind23/WindowsServer2022 upstream/jlind23/WindowsServer2022
git merge upstream/main
git push upstream jlind23/WindowsServer2022

@adam-stokes
Copy link
Contributor

@jlind23 I'd leave it open, I'll reference it when I can complete the work

@jlind23
Copy link
Contributor Author

jlind23 commented Mar 1, 2022

@adam-stokes any update there?

@mergify
Copy link
Contributor

mergify bot commented Mar 9, 2022

This pull request is now in conflict. Could you fix it @jlind23? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b jlind23/WindowsServer2022 upstream/jlind23/WindowsServer2022
git merge upstream/main
git push upstream jlind23/WindowsServer2022

@cachedout
Copy link
Contributor

@adam-stokes Could you please update this issue with the latest status? Please let me know if we need to reflect the work that needs to be done here in our sprint planning if it isn't already. Let's try to get @jlind23 unblocked if we can. :)

@jlind23
Copy link
Contributor Author

jlind23 commented Mar 31, 2022

@cachedout is there any progress that can be shared here?

@cachedout
Copy link
Contributor

@jlind23 We had to move @adam-stokes over to a more urgent project temporarily. I don't think there's any news to share on this at the moment, unfortunately.

@mdelapenya
Copy link
Contributor

I'm going to close this one in favour of #2338, which I've started a local branch for it

Please reopen if you have any consideration about it

@mdelapenya mdelapenya closed this May 3, 2022
@v1v v1v deleted the jlind23/WindowsServer2022 branch August 24, 2022 09:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-skip Skip notification from the automated backport with mergify Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants