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

Cherry-pick #12099 to 7.x: Skip Windows testing if magefile.go does not exist #12127

Closed
wants to merge 1 commit into from

Conversation

andrewkroh
Copy link
Member

Cherry-pick of PR #12099 to 7.x branch. Original message:

Changes the jenkins_ci.ps1 script to skip testing when magefile.go does not
exist. This will allow us to add projects like x-pack/winlogbeat to the test
matrix because not all branches have an x-pack/winlogbeat/magefile.go
file.

The next step will be to add x-pack/winlogbeat to the test matrix in the JJB config. And cherry-pick this into the other branches that we are actively maintaining and testing on Jenkins.

Changes the jenkins_ci.ps1 script to skip testing when magefile.go does not
exist. This will allow us to add projects like x-pack/winlogbeat to the test
matrix because not all branches have an x-pack/winlogbeat/magefile.go
file.

(cherry picked from commit cc05f62)
@ruflin
Copy link
Member

ruflin commented May 9, 2019

@andrewkroh My assumption is this backport still happens "magically"?

@andrewkroh
Copy link
Member Author

This one can't/shouldn't wait otherwise 7.x will be broken on our CI. I'm going to be changing the JJB next. But if were OK with jenkins being red until the next bulk backport then I'm good with closing this one.

@andrewkroh
Copy link
Member Author

Making one change in a Jenkinsfile on master only would be so much easier. 😢

@ruflin
Copy link
Member

ruflin commented May 9, 2019

@andrewkroh I would rather to "one more bulk backport" as I worry that this might mix up things. Being red on 7.x sounds ok to me but @urso should chime in.

++ on doing a change in a Jenkinsfile ;-)

@andrewkroh
Copy link
Member Author

No problem, I'll close this one.

@andrewkroh andrewkroh closed this May 9, 2019
@andrewkroh andrewkroh deleted the backport_12099_7.x branch January 14, 2022 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants