Skip to content

enable windows smoke test #2873

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open

enable windows smoke test #2873

wants to merge 32 commits into from

Conversation

ludfjig
Copy link
Contributor

@ludfjig ludfjig commented Aug 15, 2023 •
edited
Loading

What does this change

enable integration test running in windows

What issue does it fix

Notes for the reviewer

Checklist

  • Did you write tests?
  • Did you write documentation?
  • Did you change porter.yaml or a storage document record? Update the corresponding schema file.
  • If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️

Reviewer Checklist

  • Comment with /azp run test-porter-release if a magefile or build script was modified
  • Comment with /azp run porter-integration if it's a non-trivial PR

troy0820
troy0820 reviewed Aug 15, 2023
View reviewed changes
@ludfjig
Copy link
Contributor Author

ludfjig commented Aug 15, 2023

/azp run porter-integration

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 2873 in repo getporter/porter

@schristoff
Copy link
Member

/azp run porter-integration

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ludfjig ludfjig changed the title enable windows integration test enable windows smoke test Aug 16, 2023
ludfjig added 15 commits August 18, 2023 10:03
@ludfjig
uncomment windows build
9e04b38 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
missed a file
982dd8d 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
Trigger test
4a6f756 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
Trigger test 2
352b40b 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger 3
0ae807d 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger 4
690a934 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
update dependency
488c0e7 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trying the docker installer
56415ff 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
validate docker ps
5d5e56e 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
add PS test
f4f7541 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
add startdocker command
dbfb58b 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
6b78144 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
test gh actions
3096315 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
7f9483b 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
test2
3b3f204 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
ludfjig added 4 commits August 18, 2023 10:07
@ludfjig
fix
877d74c 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
remove newline
6233bce 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
please
18f225c 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
update go.mod
98bcf12 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
ludfjig added 13 commits August 18, 2023 10:31
@ludfjig
go.sum
26070eb 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
fix
7825bf3 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
added nat
6a062fa 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
xbuildall
8d9c1da 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
add host alias windows
64ebdf1 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
fix typo
5fbc9da 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
test selfrunner
226d5ff 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
26e629b 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
33fe342 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
6ec229d 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
a823dba 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger
6a3c4e6 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
@ludfjig
trigger2
964b480 
Signed-off-by: Ludvig Liljenberg <lliljenberg@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@troy0820 troy0820 troy0820 left review comments

@schristoff schristoff Awaiting requested review from schristoff

@sgettys sgettys Awaiting requested review from sgettys

@bdegeeter bdegeeter Awaiting requested review from bdegeeter

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ludfjig @schristoff @troy0820