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

Enable CentOS7 CI in Travis #82

Merged
merged 3 commits into from
Jan 17, 2017
Merged

Enable CentOS7 CI in Travis #82

merged 3 commits into from
Jan 17, 2017

Conversation

arm4b
Copy link
Member

@arm4b arm4b commented Jan 16, 2017

This implements #68

Enable CI for CentOS7 (kitchen-docker in TravisCI).

Allow CentOS7 build to fail in Travis until we fully implement platform support in Ansible playbooks as part of https://github.com/StackStorm/ansible-st2/milestone/2 plan.

dockerfile: .kitchen-docker/centos7/Dockerfile
# TODO: Use non-root user after updating upstream Docker image: https://github.com/StackStorm/st2-dockerfiles/issues/38
username: root
password: docker.io
Copy link
Member Author

@arm4b arm4b Jan 16, 2017

Choose a reason for hiding this comment

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

As a workaround for CentOS7 we use different approach comparing to CentOS6 #81

As @Mierdin discovered before non-root login is blocked on current CentOS7 image, so we use root as simplest approach.

Additionally, I created CentOS7 packagingtest needs re-build #38 with detailed description so we fix it in future.

@arm4b
Copy link
Member Author

arm4b commented Jan 16, 2017

So CentOS7 checks are running, SSH works, Ansible is installed and playbooks is executed: https://travis-ci.org/StackStorm/ansible-st2/jobs/192437248

CI started to fail on all existing platforms (incl. master builds) due to external dependency failure in ANXS.postgresql, which is unrelated to this PR.

I think we can merge @humblearner's PR #78 first which removes external ANXS.postgresql dependency (just in time!).

@arm4b
Copy link
Member Author

arm4b commented Jan 17, 2017

Rebased on top of master which removes external role ANXS.postgresql dependency and the build is green again.

@arm4b arm4b added RFR and removed WIP labels Jan 17, 2017
Copy link
Contributor

@humblearner humblearner left a comment

Choose a reason for hiding this comment

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

👍 +1

@arm4b arm4b merged commit a7c3dad into master Jan 17, 2017
@arm4b arm4b deleted the feature/centos7-ci branch January 17, 2017 19:58
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.

3 participants