GEODE-6054: Move Windows CI workers to Server 2016 and VS 2017#439
Merged
pdxcodemonkey merged 35 commits intoapache:developfrom Jan 28, 2019
Merged
GEODE-6054: Move Windows CI workers to Server 2016 and VS 2017#439pdxcodemonkey merged 35 commits intoapache:developfrom
pdxcodemonkey merged 35 commits intoapache:developfrom
Conversation
Co-authored-by: Ernest Burghardt <eburghardt@pivotal.io>
Co-authored-by: Ernest Burghardt <eburghardt@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io>
Co-authored-by: Matt Reddington <mreddington@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
- increase instance root volume size to 100GB Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
… image Co-authored-by: Mike Martell <mmartell@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
…p script where missing Co-authored-by: Jacob Barrett <jbarrett@pivotal.io> Co-authored-by: Ivan Godwin <igodwin@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io>
Co-authored-by: Mike Martell <mmartell@pivotal.io>
- add 60s delay before install-dependencies Co-authored-by: Mike Martell <mmartell@pivotal.io>
- initialize scripts added to each stage
Member
|
I don't see any readme/building dot md changes, is that something that needs updating also? |
Codecov Report
@@ Coverage Diff @@
## develop #439 +/- ##
===========================================
+ Coverage 72.23% 72.26% +0.03%
===========================================
Files 654 654
Lines 52351 52351
===========================================
+ Hits 37814 37830 +16
+ Misses 14537 14521 -16
Continue to review full report at Codecov.
|
igodwin
requested review from
echobravopapa,
jake-at-work,
moleske and
pdxcodemonkey
pdxcodemonkey
approved these changes
Jan 28, 2019
pdxcodemonkey
pushed a commit
that referenced
this pull request
Feb 11, 2019
* Build with Visual Studio 2017 runtime. * Fix up VS2015 dependencies in 3rd-party OSS * Add 2017 provisioner file * Use default version of OpenSSH * Added script to fix ssh-user permissions * Swith from choco to vs bootstrapper * Execute Repair-AuthorizedKeyPermission on boot * Fixed path to OpenSSHUtils * Wait for install to finish, before exit * Merging init-ssh and install-ssh to avoid permission issues * Point to proper vcvarsall.bat location * added windows 10 sdk to package ids * Mark ThinClientPoolTestsN as FLAKY * Up SSD size for build-windows AMI, to agree with new base image * GEODE-6054: Run EC2Launch script at end of packer scripts, add cleanup script where missing * installing .NET 3.5 for nunit * Add component IDs for VisualStudio2017 install * Use two part install of visual studio * Switch to direct aka download of vs bootstrapper * GEODE-6054: Replace EC2Config with EC2Launch in base AMI step * Refactor initialization process * add 60s delay before install-dependencies Co-authored-by: Ivan Godwin <igodwin@pivotal.io> Co-authored-by: Mike Martell <mmartell@pivotal.io> Co-authored-by: Ernie Burghardt <eburghardt@pivotal.io> Co-authored-by: Jake Barrett <jbarrett@pivotal.io> Co-authored-by: Matt Reddington <mreddington@pivotal.io>
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
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Windows CI workers have been updated to the latest Windows Server 2016 release, and now use Visual Studio 2017 for builds. A lot of work done to get the latest version of openssh (from Chocolatey) to work on Server 2016, as well as migrate EC2 initialization and cleanup scripts to work with current methods.