-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add support for aws_ec2_vpc_net role in Linchpin #907
Add support for aws_ec2_vpc_net role in Linchpin #907
Conversation
ci-linchpin-messageBus-trigger build #772: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/772/ |
ci-linchpin-messageBus-trigger build #773: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/773/ |
ci-linchpin-messageBus-trigger build #774: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/774/ |
[test] |
1 similar comment
[test] |
9953a93
to
3d80731
Compare
rebased to fix conflicts with develop branch |
ci-linchpin-messageBus-trigger build #785: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/785/ |
…even on empty provision
ci-linchpin-messageBus-trigger build #786: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/786/ |
ci-linchpin-messageBus-trigger build #788: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/788/ |
the unit tests are failing because we don't have gcc. :-( We will need to add gcc to the container |
@p3ck How come it worked before then ?? |
@p3ck : we do have gcc in all the dockerfiles |
The unit tests are executed via HDSL in the ansible-executor. We may have to actually provision a resource to run it. I don't know why it worked before. Did you add/update any Requires in the latest patch? |
@p3ck : I added boto3 in requirements.txt. However , That's in previous commits which worked fine for unit tests. |
[test] |
ci-linchpin-messageBus-trigger build #791: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/791/ |
[test] |
ci-linchpin-messageBus-trigger build #792: SUCCESS: https://jenkins-continuous-infra.cloud.continuous-infra.upshift.redhat.com/job/ci-linchpin-messageBus-trigger/792/ |
[merge] |
$subject
example:
fixes: #827
should be merged after #894