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

(RE-12859) Migrate from enterprise.delivery.puppetlabs.net to Artifactory #171

Merged

Conversation

mwaggett
Copy link
Contributor

This commit replaces usage of enterprise.delivery.puppetlabs.net (aka saturn)
with Artifactory so that we can ultimately decommission it.

…tory

This commit replaces usage of enterprise.delivery.puppetlabs.net (aka saturn)
with Artifactory so that we can ultimately decommission it.
@mwaggett
Copy link
Contributor Author

I'm not totally sure how pe_dir is used, so I don't know if this will Just Work™ or not.

@nmburgan
Copy link
Contributor

I think this will work, but we'll need to get everything from 2016 forward into artifactory, or else those tests that use pre-Irving versions will fail.

@mwaggett
Copy link
Contributor Author

@nmburgan Do we need to be testing anything pre-Irving?? I know we have the one 2016.4 straggler, but besides that?

@nmburgan
Copy link
Contributor

@mwaggett
Copy link
Contributor Author

oh gross.

well those would be coming from archives/releases anyway, right? Those should already all be in Artifactory: https://artifactory.delivery.puppetlabs.net/artifactory/webapp/#/artifacts/browse/tree/General/generic_enterprise__local/archives/releases

@nmburgan
Copy link
Contributor

Oh right, good point :)

@mwaggett mwaggett marked this pull request as ready for review October 14, 2019 21:19
@nmburgan
Copy link
Contributor

I'm testing a fresh 2018.1.10 install and a 2016.4.15 -> 2018.1.10 now using this change.

@nmburgan
Copy link
Contributor

We should put a DNM (or I guess just not cut a gem?) until after the Lovejoy release.

@mwaggett
Copy link
Contributor Author

yeah, I have no idea how this gets released, but +1 to not shipping before Lovejoy, whether that means waiting to merge or waiting to release

@nmburgan
Copy link
Contributor

Verified that setting pe_version and pe_upgrade_version environment variables and then running bundle exec beaker-hostgenerator centos7-64mcda --hypervisor=vmpooler outputs a host file that points to artifactory for pe_dir and pe_upgrade_dir. This is what CI does to generate the file.

Verified that that the install and upgrade noted above worked successfully pulling the tarballs from artifactory.

Copy link
Contributor

@nmburgan nmburgan left a comment

Choose a reason for hiding this comment

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

Both install and upgrade passed fine, so I'm 👍 to make this change after Lovejoy is out the door.

Copy link
Contributor

@kevpl kevpl left a comment

Choose a reason for hiding this comment

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

lgtm 👍. Let's wait to merge until the release happens, so that we can do other things if we need to in the meantime. Applied label for this.

@mwaggett
Copy link
Contributor Author

mwaggett commented Nov 8, 2019

@nmburgan @kevpl This should be good to go in, now that Lovejoy is done. No idea how this gets released though..

@nmburgan
Copy link
Contributor

nmburgan commented Nov 8, 2019

Turns out we're going to need a quick 2019.2.1. Let's wait until that goes out the door.

@mwaggett
Copy link
Contributor Author

okay how about now??

@kevpl kevpl merged commit d4a4bc2 into voxpupuli:master Nov 15, 2019
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.

4 participants