Closed jordimassaguerpla closed 5 years ago
This needs to go together with https://github.com/kubic-project/caasp-container-manifests/pull/221
@MaximilianMeister : will it be ok for you that we merge both Requests? I don't think tests can pass separately...
there is a an hackish way to test 2 prs (from 2 diff repos) before merging. you can do this in the repo jenkins-library
with a PR and from there specify your salt and other repos, i think we could do it before merging.
@MalloZup that sounds great! Could you do that for me? Unfortunately I am not familiar enough with the CI setup to do this hack ... I would really appreciate since this is the last thing needed for the next release.
@jordimassaguerpla this should work here https://github.com/SUSE/caasp-jenkins-library/pull/276
@MaximilianMeister i have done the Pipeline for the 2 prs, is running now: http://jenkins.caasp.suse.net/blue/organizations/jenkins/jenkins-library.integration/detail/PR-276/5/pipeline
if green we can merge. I have created follow-up issue
@MaximilianMeister i have done the Pipeline for the 2 prs, is running now: http://jenkins.caasp.suse.net/blue/organizations/jenkins/jenkins-library.integration/detail/PR-276/5/pipeline
if green we can merge. I have created follow-up issue
@MalloZup where is the follow up issue?
it is in between comments @hwoarang :gear: it is here: https://github.com/kubic-project/salt/issues/748
I cherry-picked into https://github.com/kubic-project/salt/pull/750
Signed-off-by: Jordi Massaguer Pla jmassaguerpla@suse.de