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

add pwsh step #122

Merged
merged 4 commits into from
Nov 1, 2019
Merged

add pwsh step #122

merged 4 commits into from
Nov 1, 2019

Conversation

jetersen
Copy link
Member

@jetersen jetersen commented Oct 7, 2019

downstream of jenkinsci/durable-task-plugin#111

Hmm, should I add docker fixture or testcontainers or hope that our Windows server 2019 have pwsh?

@dwnusbaum dwnusbaum requested review from caldwell, dwnusbaum and car-roll and removed request for caldwell October 7, 2019 16:20
Copy link
Collaborator

@car-roll car-roll left a comment

Choose a reason for hiding this comment

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

I'm fine with either

@gabloe
Copy link
Contributor

gabloe commented Oct 8, 2019

@Casz the Windows Server Images likely do not have open powershell (pwsh) installed

@gabloe
Copy link
Contributor

gabloe commented Oct 8, 2019

This looks good to me.

@jetersen
Copy link
Member Author

jetersen commented Oct 8, 2019

I think the testing done in upstream should suffice, if anyone disagree. I can copy the same test here 😓

@jetersen
Copy link
Member Author

I updated it to use the released version, so a mighty release of this would be appreciated @dwnusbaum @car-roll 🙏

@dwnusbaum
Copy link
Member

@Casz FYI, I filed jenkins-infra/update-center2#305 to suspend distribution of durable-task 1.31 due to some issues on various platforms from a separate change, so we don't want to merge/release this with a dependency on 1.31. We're still trying to decide what to do with that issue, we might end up releasing 1.32 that reverts/disables the problematic change and then we could update to that here, or we might try to fix the issues with jenkinsci/durable-task-plugin#114 and then release 1.32.

@jetersen
Copy link
Member Author

😭

@car-roll
Copy link
Collaborator

so 1.32 has been released and #123 has the 1.32 change as well as a changelog that covers this PR. Since ci.jenkins.io is down, I don't know if you just want to run your tests locally with 1.32 to make sure everything is okay. Would that be good enough to merge this in @dwnusbaum ?

@jetersen
Copy link
Member Author

Here is my basic test that demonstrate it works:

Pipeline:

node {
    pwsh 'Write-Host "Hello from $($PSVersionTable.PSVersion) $($PSVersionTable.PSEdition)"'
    powershell 'Write-Host "Hello from $($PSVersionTable.PSVersion) $($PSVersionTable.PSEdition)"'
}

Run:

Started by user unknown or anonymous
Running in Durability level: MAX_SURVIVABILITY
[Pipeline] Start of Pipeline
[Pipeline] node
Running on Jenkins in C:\code\workflow-durable-task-step-plugin\work\workspace\test
[Pipeline] {
[Pipeline] pwsh
Hello from 6.2.3 Core
[Pipeline] powershell
Hello from 5.1.19008.1 Desktop
[Pipeline] }
[Pipeline] // node
[Pipeline] End of Pipeline
Finished: SUCCESS

image

@jetersen
Copy link
Member Author

I can add a test case @car-roll if you want :)

@jetersen
Copy link
Member Author

@slide hmm the new Windows 2019 servers do they have powershell core or do I have to put it in docker?

@slide
Copy link
Member

slide commented Oct 28, 2019

They do not have powershell core right now.

@jetersen
Copy link
Member Author

@slide I don't see any docs for buildPlugin on how to use Windows Server 2019 for using Windows Docker 😅

@slide
Copy link
Member

slide commented Oct 28, 2019

@Casz good question, this is the first time this has come up so far. We may need to enhance

@car-roll
Copy link
Collaborator

car-roll commented Oct 30, 2019

So we are not at 1.33 for durable-task. That should resolve the issues that were introduced in 1.31. So far no other complaints, so you can go ahead and update this dependency again (sorry 😞 ). I think maybe another day with no other major issues popping up, we can merge and release

@jetersen
Copy link
Member Author

jetersen commented Nov 1, 2019

Would appreciate a release 🙏

@car-roll
Copy link
Collaborator

car-roll commented Nov 1, 2019

sorry for all the delays @Casz 😢 Took some time to make sure durable-task 1.33 didn't introduce any more problems. I'm merging this now.

@car-roll car-roll merged commit f33d4e1 into jenkinsci:master Nov 1, 2019
@jetersen jetersen deleted the feature/pwshStep branch November 1, 2019 16:50
@jetersen
Copy link
Member Author

jetersen commented Nov 1, 2019

No, thank you! Sorry for being so pushy 😓

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants