Revert "Test with Katello"

[parthaa]

This reverts commit b0ae4c6.

This prevents this error

[!] There was an error parsing `Gemfile`: You cannot specify the same gem twice coming from different sources.
You specified that katello (>= 0) should come from https://github.com/Katello/katello.git and source at `../katello`
. Bundler cannot continue.

 #  from /home/vagrant/foreman/Gemfile:59
 #  -------------------------------------------
 #    puts "looking at '#{bundle}'"
 >    instance_eval(Bundler.read_file(bundle))
 #  end
 #  -------------------------------------------

[chris1984]

@ofedoren this reverts the katello bundle install error Partha posted. Do you have a suggestion to get the tests green again?

[ekohl]

I think this is tricky. We need this for CI and I don't see any other way. But then theforeman/puppet-katello_devel@6d42c4c combines all the different gemfile.d entries and that creates a conflict.

[ofedoren]

Do you have a suggestion to get the tests green again?

Either have this commit or release Katello with Zeitwerk changes, not sure if there is another way...

[ekohl]

Perhaps one option is to change this into a group:

group :foreman_virt_who_configure_deps do
  gem 'katello', github: 'Katello/katello', branch: 'master'
end

Then you might be able to use bundle config without foreman_virt_who_configure_deps to skip it, but I recall Bundler always evaluates all dependencies so it may not work.

Another ugly option is to hide it behind some ENV var, which is also ugly but perhaps less so.

[ekohl]

I opened #202 which can work, but has some caveats.

[ehelms]

I propose we do not revert the change but instead change how we handle the devel environment -- theforeman/puppet-katello_devel#307