FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add Ubuntu SSH remoting tests CI by PaulHigin · Pull Request #12033 · PowerShell/PowerShell · GitHub

Add Ubuntu SSH remoting tests CI - #12033

Merged
Travis Plunk (TravisEz13) merged 33 commits into
PowerShell:masterfrom
PaulHigin:add-sshremoting-tests-ci
Mar 11, 2020
Merged

Add Ubuntu SSH remoting tests CI#12033
Travis Plunk (TravisEz13) merged 33 commits into
PowerShell:masterfrom
PaulHigin:add-sshremoting-tests-ci

Conversation

Paul Higinbotham (PaulHigin) commented Mar 5, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

PR Summary

This PR adds CI for SSH remoting tests for Ubuntu platforms

PR Context

This PR includes basic CI for installing PowerShell and setting up SSH remoting on a Ubuntu platform, then running SSHRemoting tests.

PR Checklist

Comment thread .vsts-ci/sshremoting-tests.yml Outdated
ghost added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 5, 2020
Paul Higinbotham (PaulHigin) changed the title [WIP] Add Ubuntu SSH remoting tests CI Add Ubuntu SSH remoting tests CI Mar 10, 2020
ghost removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 10, 2020

Copy link
Copy Markdown
Contributor Author

Removed WIP, as tests are running and passing.

Comment thread .vsts-ci/sshremoting-tests.yml Outdated
Comment thread .vsts-ci/sshremoting-tests.yml Outdated
Comment thread .vsts-ci/sshremoting-tests.yml Outdated
ghost added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 10, 2020
Co-Authored-By: Travis Plunk <travis.plunk@microsoft.com>
ghost removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 10, 2020
Co-Authored-By: Travis Plunk <travis.plunk@microsoft.com>
Co-Authored-By: Travis Plunk <travis.plunk@microsoft.com>

Copy link
Copy Markdown
Contributor Author

Travis Plunk (@TravisEz13) Can this PR be merged?

Comment thread .vsts-ci/sshremoting-tests.yml Outdated
Comment thread .vsts-ci/sshremoting-tests.yml Outdated
ghost added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 11, 2020

Copy link
Copy Markdown
Member

PoshChan-Bot (@PoshChan) Please remind me in 1 hour

Copy link
Copy Markdown
Collaborator

Travis Plunk (@TravisEz13), this is the reminder you requested 1 hour ago

Copy link
Copy Markdown
Member

I setup the new CI build
https://dev.azure.com/powershell/PowerShell/_build?definitionId=85&_a=summary

Paul Higinbotham (PaulHigin) deleted the add-sshremoting-tests-ci branch March 11, 2020 22:52
Aditya Patwardhan (adityapatwardhan) added CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log and removed Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept labels Mar 24, 2020

Copy link
Copy Markdown

🎉v7.1.0-preview.1 has been released which incorporates this pull request.:tada:

Handy links:

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
* Add SSH remoting CI

* Fix typo

* Add install git to Ubuntu CI

* Update .vsts-ci/sshremoting-tests.yml

Co-Authored-By: Aditya Patwardhan <adityap@microsoft.com>

* Fix install git 1

* Add missing tools module import

* Change ubuntu service restart

* Update ssh install

* fix module path

* fix module path

* change module import

* Add tracing

* Add service start retry

* Fix service restart

* Fix options restore

* Fix Restore-PSOptions path

* Fix Pester test output

* fix typo

* Fix test output path

* Debug 1

* Debug 2

* Debug 3

* Change results path

* Fix result publish to use build artifacts directory

* Add more New-PSSession tests

* Remove User test

* Remove env:USER

* Add API tests

* Fix type for Subsytem API test

* Update .vsts-ci/sshremoting-tests.yml

Co-Authored-By: Travis Plunk <travis.plunk@microsoft.com>

* Update .vsts-ci/sshremoting-tests.yml

Co-Authored-By: Travis Plunk <travis.plunk@microsoft.com>

* Update .vsts-ci/sshremoting-tests.yml

Co-Authored-By: Travis Plunk <travis.plunk@microsoft.com>

* Apply suggestions from code review

Co-authored-by: Aditya Patwardhan <adityap@microsoft.com>
Co-authored-by: Travis Plunk <travis.plunk@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL