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

#2928: Replace hiera version 3 with version 5 #2929

Merged
merged 4 commits into from
Mar 5, 2017
Merged

Conversation

jeff1evesque
Copy link
Owner

Resolves #2928.

@jeff1evesque jeff1evesque added this to the 0.6 milestone Mar 5, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 48.801% when pulling a08adbc on feature-2928 into 64b7bcf on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 48.801% when pulling 55584a8 on feature-2928 into 64b7bcf on master.

@jeff1evesque
Copy link
Owner Author

jeff1evesque commented Mar 5, 2017

We've manually built our application via vagrant up, and noticed our earlier warnings no longer displayed:

$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Running triggers before up...
[TRACEBACK-OMITTED]
==> default: Running provisioner: puppet...
==> default: Running Puppet with environment vagrant...
==> default: stdin: is not a tty
==> default: Warning: Unknown variable: 'python::params::provider'. at /tmp/vagrant-puppet/modules-3c00084ae9953309c24252b2dd2bf5cd/python/manifests/init.pp:56:25
==> default: Warning: Unknown variable: 'python::params::source'. at /tmp/vagrant-puppet/modules-3c00084ae9953309c24252b2dd2bf5cd/python/manifests/init.pp:61:25
==> default: Notice: Compiled catalog for ubuntu-14.04 in environment vagrant in 1.65 seconds
==> default: Notice: /Stage[main]/Webserver::Service/File[/etc/init/start_gunicorn.conf]/ensure: defined content as '{md5}95d861b3c916da7f8e76427279e608af'
==> default: Notice: /Stage[main]/Package::Gunicorn/Package[gunicorn]/ensure: created
==> default: Notice: /Stage[main]/Webserver::Start/Service[start_gunicorn]/ensure: ensure changed 'stopped' to 'running'
==> default: Notice: /Stage[main]/Nginx::Package::Debian/Apt::Source[nginx]/Apt::Key[Add key: 573BFD6B3D8FBC641079A6ABABF5BD827BD9BF62 from Apt::Source nginx]/Apt_key[Add key: 573BFD6B3D8FBC641079A6ABABF5BD827BD9BF62 from Apt::Source nginx]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Package::Debian/Apt::Source[nginx]/Apt::Setting[list-nginx]/File[/etc/apt/sources.list.d/nginx.list]/ensure: defined content as '{md5}44b89c7233b93da7e096f3f51334df58'
==> default: Notice: /Stage[main]/Apt::Update/Exec[apt_update]: Triggered 'refresh' from 1 events
==> default: Notice: /Stage[main]/Nginx::Package::Debian/Package[nginx]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/etc/nginx/conf.stream.d]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/etc/nginx/conf.mail.d]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/var/nginx]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/var/log/nginx]/group: group changed 'adm' to 'root'
==> default: Notice: /Stage[main]/Nginx::Config/File[/var/nginx/client_body_temp]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/var/nginx/proxy_temp]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/etc/nginx/streams-enabled]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/etc/nginx/streams-available]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Config/File[/etc/nginx/nginx.conf]/content: content changed '{md5}a85dcbc5367783f3d42d0b7eab22ea50' to '{md5}de983fff1c3d7515a8100b89ed6ab532'
==> default: Notice: /Stage[main]/Webserver::Service/Nginx::Resource::Vhost[machine-learning.com]/Concat[/etc/nginx/sites-available/machine-learning.com.conf]/File[/etc/nginx/sites-available/machine-learning.com.conf]/ensure: defined content as '{md5}5937732bcc4cee3f211f266c2e3ccbb1'
==> default: Notice: /Stage[main]/Webserver::Service/Nginx::Resource::Vhost[machine-learning.com]/File[machine-learning.com.conf symlink]/ensure: created
==> default: Notice: /Stage[main]/Nginx::Service/Service[nginx]: Triggered 'refresh' from 1 events
==> default: Notice: Applied catalog in 24.89 seconds

@coveralls
Copy link

Coverage Status

Coverage remained the same at 48.801% when pulling 51089c3 on feature-2928 into 64b7bcf on master.

@jeff1evesque
Copy link
Owner Author

We notice the application still executes as expected:

prediction

@jeff1evesque jeff1evesque merged commit 47fd5cb into master Mar 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants