-
Notifications
You must be signed in to change notification settings - Fork 29
update haproxy tag to the version for caasp4 #747
Conversation
This needs to go together with SUSE/caasp-container-manifests#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 |
@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 |
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.
I'm ok for merging as is, if we follow up on the pillar setting in another PR
@MaximilianMeister i have done the Pipeline for the 2 prs, is running now: if green we can merge. I have created follow-up issue |
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.
🐷 wait for the other pipeline otherwise ok
@MalloZup where is the follow up issue? |
Signed-off-by: Jordi Massaguer Pla <jmassaguerpla@suse.de>
bd9db97
to
8c97ff9
Compare
I cherry-picked into #750 |
Signed-off-by: Jordi Massaguer Pla jmassaguerpla@suse.de