-
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
Jenkins release job #594
Jenkins release job #594
Conversation
ci-linchpin-messageBus-trigger build #128: SUCCESS: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/128/ |
ci-linchpin-messageBus-trigger build #130: FAILURE: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/130/ |
[test] |
ci-linchpin-messageBus-trigger build #131: FAILURE: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/131/ |
Adjusted to only tag when it doesn't already exist.
7889e1c
to
ce5b949
Compare
ci-linchpin-messageBus-trigger build #178: SUCCESS: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/178/ |
Minimal set of changes to enable support for remote libvirt hypervisors.
``` TASK [ovirt : Provisioning resource definitions of current group] *********************************************** task path: /home/user/venv/lib/python2.7/site-packages/linchpin/provision/roles/ovirt/tasks/provision_resource_gr yml:33 fatal: [localhost]: FAILED! => { "msg": "template error while templating string: unexpected ')'. String: {{ res_grp['resource_definitions']) } } ```
Please note, the JenkinsfileRelease is using branches in my repository. When this is reviewed, note the comments as they will be updated before merging. |
ci-linchpin-messageBus-trigger build #209: FAILURE: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/209/ |
ci-linchpin-messageBus-trigger build #210: FAILURE: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/210/ |
[test] |
ci-linchpin-messageBus-trigger build #211: SUCCESS: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/211/ |
set retries to 10 for pip download adjust the error handling at the end to properly verify version
…jenkins_release_job
ci-linchpin-messageBus-trigger build #212: SUCCESS: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/212/ |
Since there will likely be further testing, the testing values stayed, but commented out. That way, it will be simple to run more testing in the future.
ci-linchpin-messageBus-trigger build #213: SUCCESS: https://jenkins-continuous-infra.cloud.rhel8.upshift.engineering.redhat.com/job/ci-linchpin-messageBus-trigger/213/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm - just check on the openstack image before merge
@@ -16,7 +16,7 @@ resource_groups: | |||
- name: "{{ distro | default('') }}frontend" | |||
role: os_server | |||
flavor: m1.small | |||
image: Fedora-Cloud-Base-26-compose-latest | |||
image: CentOS-7-x86_64-GenericCloud-1612 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this need to be switched back? I know sk had to change this because the CentOS image was missing? Or do I have it backwards?
I cherry-picked his commit. I suspect there will be no merge conflict with
that...
…On Fri, Jun 15, 2018, 12:54 Bill Peck ***@***.***> wrote:
***@***.**** approved this pull request.
lgtm - just check on the openstack image before merge
------------------------------
In docs/source/examples/workspace/topologies/os-server-new.yml
<#594 (comment)>
:
> @@ -16,7 +16,7 @@ resource_groups:
- name: "{{ distro | default('') }}frontend"
role: os_server
flavor: m1.small
- image: Fedora-Cloud-Base-26-compose-latest
+ image: CentOS-7-x86_64-GenericCloud-1612
Does this need to be switched back? I know sk had to change this because
the CentOS image was missing? Or do I have it backwards?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#594 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAFc9vwdkWo3Rz0uq-CwYbRUfg-SWvhPks5t9ALfgaJpZM4UHoFu>
.
|
[merge] |
[merge]
…On Mon, Jun 18, 2018, 07:45 paas-bot ***@***.***> wrote:
Merged #594 <#594>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#594 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAFc9iYDNuUsB0ZZOYijJ-n-Ig81hVfsks5t967XgaJpZM4UHoFu>
.
|
Add cd-linchpin-release config.xml
Update JenkinsfileRelease job
Make Dockerfile match what is in ci-pipeline for jenkins slave