Katello-nightly-rpm-pipeline 45 failed

Katello nightly pipeline failed:

https://ci.theforeman.org/job/katello-nightly-rpm-pipeline/45/

This is because V2 API ping call support is merged in core, but the relevant changes to katello, hammer-cli-foreman and hammer-cli-katello are not.

@lzap @ofedoren what is needed to get the rest of the changes in quickly?
Or should that change maybe not be in 1.22 at all? @tbrisker

Everything is almost ready:
Katello: https://github.com/Katello/katello/pull/7751
hammer-cli-foreman: https://github.com/theforeman/hammer-cli-foreman/pull/394
hammer-cli-katello: https://github.com/Katello/hammer-cli-katello/pull/596

I’m going to review those today and if someone will be interested in testing and/or reviews, I think it can be merged today or on Monday. Also, I’m not insisting on having it in 1.22 if it causes any inconvenience.

Please revert the core change and restore it after branching. We are in stabilization period and need to have green nightlies when branching, this appears to require quite a lot of changes.

Sorry about that. Missed that name clash, normally it would be pretty safe but this is actually move of code from Katello to Core. Reverting, let’s do it later.

https://github.com/theforeman/foreman/pull/6674

1 Like

@ofedoren please create a new PR, let’s just wait after branching. My bad.

Thanks @lzap!

Sure :slight_smile: Thanks.

This topic was automatically closed 7 days after the last reply. New replies are no longer allowed.