Skip to content

Conversation

@adamruzicka
Copy link

  • Update smart_proxy_dynflow to 0.6.0
  • Update smart_proxy_dynflow to 0.6.1

@Odilhao Odilhao merged commit c6ce631 into theforeman:deb/3.1 Dec 6, 2021
@adamruzicka adamruzicka deleted the deb/3.1-sp-dynflow branch December 7, 2021 08:52
@shr-project
Copy link

shr-project commented Dec 9, 2021

Does this depend on some other PR which didn't make it into 3.1 release?

ruby-dynflow in debian seems to be still 1.5.0 version:
https://github.com/theforeman/foreman-packaging/tree/deb/3.1/dependencies/focal/dynflow
which prevents this package to be installed:

ruby-smart-proxy-dynflow : Depends: ruby-dynflow (>= 1.6.0) but 1.5.0-1 is to be installed

In rpm packaging it's ok, because
https://github.com/theforeman/foreman-packaging/blob/298c1ff374282cc78d13dc76fb7de443b10034ce/packages/foreman/rubygem-dynflow/rubygem-dynflow.spec
is version 1.6.1.

@evgeni
Copy link
Member

evgeni commented Dec 9, 2021

Yeah.

@adamruzicka I recall we added new msgpack for this in develop, but I don't recall if we ever back ported these changes to 3.1?

@shr-project
Copy link

I've found some older PR which might be related #7115

@adamruzicka
Copy link
Author

Iirc, this needs dynflow >= 1.6.0, dynflow >= 1.6.0 needs msgpack. Msgpack was merged to develop, but probably not backported yet. dynflow-1.6.1 was not even merged to develop, let alone picked to 3.1

@evgeni
Copy link
Member

evgeni commented Dec 10, 2021

Iirc, this needs dynflow >= 1.6.0, dynflow >= 1.6.0 needs msgpack. Msgpack was merged to develop, but probably not backported yet. dynflow-1.6.1 was not even merged to develop, let alone picked to 3.1

correct, which makes it weird that this one was merged, but it was.

given dynflow 1.6.z is is in rpm 3.1, I'll make sure it also lands there for debian

@shr-project
Copy link

given dynflow 1.6.z is is in rpm 3.1, I'll make sure it also lands there for debian

thanks, please do, the current state prevents to install ansible plugin for 3.1 which depends on all this

@shr-project
Copy link

shr-project commented Dec 10, 2021

Thanks for fix, now apt update from 3.1 plugins was successful, unfortunately it still doesn't work well in runtime as smart_proxy_remote_execution_ssh seem to require ruby-mqtt (reported in #7355 (comment))

@evgeni
Copy link
Member

evgeni commented Dec 10, 2021

Thanks for the further report, this will have to wait till Monday though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants