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

Update based on voxpupuli/modulesync_config 0.16.3 #382

Merged
merged 9 commits into from
Dec 7, 2016
Merged

Update based on voxpupuli/modulesync_config 0.16.3 #382

merged 9 commits into from
Dec 7, 2016

Conversation

dhoppe
Copy link
Member

@dhoppe dhoppe commented Oct 20, 2016

Please merge. This is an requirement for the Coveralls badge at README.md.

docker_image_commands:
- 'yum install -y crontabs tar wget openssl sysvinit-tools iproute which initscripts'
- 'rm -rf /var/run/network/*'
- 'yum install -y crontabs tar wget'
Copy link
Member

Choose a reason for hiding this comment

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

we shoukd probably update the nodeset in modulesync to also contain the other packages (of they are needed)

platform: el-6-x86_64
hypervisor: docker
hypervisor : docker
Copy link
Member

Choose a reason for hiding this comment

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

that may be bad

@bastelfreak
Copy link
Member

before we merge this we should readd the docker tests

@dhoppe
Copy link
Member Author

dhoppe commented Oct 20, 2016

I tried to add some Docker tests, but Beaker 3.1.0 does not support Ruby 2.1.9 and even if I pin it to Beaker 2.51.0 it fails with several error messages. However, a run time of 20 minutes is not acceptable.

.travis.yml:
  extras:
  - rvm: 2.1.9
    env: BEAKER_VERSION="~> 2.51" PUPPET_VERSION="~> 3.0" STRICT_VARIABLES="yes" CHECK=acceptance BEAKER_debug=true BEAKER_set=docker/centos-7 PUPPET_INSTALL_TYPE=foss PUPPET_INSTALL_VERSION=3.8.7
    services: docker
    sudo: required
    bundler_args: --without development
  - rvm: 2.1.9
    env: BEAKER_VERSION="~> 2.51" PUPPET_VERSION="~> 3.0" STRICT_VARIABLES="yes" CHECK=acceptance BEAKER_debug=true BEAKER_set=docker/ubuntu-14.04 PUPPET_INSTALL_TYPE=foss PUPPET_INSTALL_VERSION=3.8.7
    services: docker
    sudo: required
    bundler_args: --without development
  - rvm: 2.3.1
    env: BEAKER_VERSION="~> 2.51" PUPPET_VERSION="~> 4.0" CHECK=acceptance BEAKER_debug=true BEAKER_set=docker/centos-7 PUPPET_INSTALL_TYPE=agent PUPPET_INSTALL_VERSION=1.7.1
    services: docker
    sudo: required
    bundler_args: --without development
  - rvm: 2.3.1
    env: BEAKER_VERSION="~> 2.51" PUPPET_VERSION="~> 4.0" CHECK=acceptance BEAKER_debug=true BEAKER_set=docker/debian-8 PUPPET_INSTALL_TYPE=agent PUPPET_INSTALL_VERSION=1.7.1
    services: docker
    sudo: required
    bundler_args: --without development
  - rvm: 2.3.1
    env: BEAKER_VERSION="~> 2.51" PUPPET_VERSION="~> 4.0" CHECK=acceptance BEAKER_debug=true BEAKER_set=docker/ubuntu-14.04 PUPPET_INSTALL_TYPE=agent PUPPET_INSTALL_VERSION=1.7.1
    services: docker
    sudo: required
    bundler_args: --without development
  - rvm: 2.3.1
    env: BEAKER_VERSION="~> 2.51" PUPPET_VERSION="~> 4.0" CHECK=acceptance BEAKER_debug=true BEAKER_set=docker/ubuntu-16.04 PUPPET_INSTALL_TYPE=agent PUPPET_INSTALL_VERSION=1.7.1
    services: docker
    sudo: required
    bundler_args: --without development

@roidelapluie
Copy link
Member

@dhoppe it is not really adding. We have the tests for a long time already

@dhoppe
Copy link
Member Author

dhoppe commented Oct 20, 2016

@roidelapluie The last successful build used Beaker 2.51.0, is 28 days ago and took 1 hour and 46 minutes.

I just want to get this merged and start working on the readme and code coverage of the rspec tests.

If we want to use acceptance tests properly, we need our own hardware or do this locally. Maybe someone else knows how to create the proper docker_image_comands and fix the configuration for Beaker 3.1.0.

@dhoppe
Copy link
Member Author

dhoppe commented Nov 4, 2016

I think the acceptance tests are failing because of the type at the nodesets. Puppet 3 needs the type set to foss and Puppet 4 to aio.

@roidelapluie
Copy link
Member

travis fails

Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
@dhoppe dhoppe changed the title Update based on voxpupuli/modulesync_config 0.14.1 Update based on voxpupuli/modulesync_config 0.16.3 Dec 6, 2016
@dhoppe
Copy link
Member Author

dhoppe commented Dec 6, 2016

@roidelapluie I had to restart one of the build steps, because it was running for hours and just did a modulesync against 0.16.3.

@dhoppe dhoppe merged commit b4a1d32 into voxpupuli:master Dec 7, 2016
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.

None yet

4 participants