Katello Rails 5 PR Testing

Katello developers may notice a 'rails-5' status pop up on pull requests. I am working to add testing against Rails 5 stack. This currently fails so it can safely be ignored. This thread will get an update when the testing has been officially implemented and thus should be a gate for new PRs.

··· --
Eric D. Helms
Red Hat Engineering
Updates. The test job is currently working as expected for Katello and failing with an error a developer will need to look into. See traceback below.

NOTE: For any other plugin maintainers reading this if you'd like a dedicated Rails 5 PR test job let me know and we can work to add one similar to what I've done for Katello.

class AddDockerManifestList < ActiveRecord::Migration[4.2]
/usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/bin/rake:22) == 20171010172724 AddDockerManifestList: migrating ============================ -- create_table(:katello_docker_manifest_lists, {})
   -> 0.0028s
-- create_table(:katello_repository_docker_manifest_lists, {})    -> 0.0020s
-- create_table(:katello_docker_manifest_list_manifests, {})    -> 0.0019s
-- remove_foreign_key(:katello_docker_tags, :docker_manifest) rake aborted!
StandardError: An error has occurred, this and all later migrations canceled:

Table 'katello_docker_tags' has no foreign key for docker_manifest /usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/gems/activerecord-5.0.6/lib/active_record/connection_adapters/abstract/schema_statements.rb:970:in `foreign_key_for!'
/usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/gems/activerecord-5.0.6/lib/active_record/connection_adapters/abstract/schema_statements.rb:940:in `remove_foreign_key'
/usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/gems/activerecord-5.0.6/lib/active_record/migration.rb:846:in `block in method_missing'
/usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/gems/activerecord-5.0.6/lib/active_record/migration.rb:815:in `block in say_with_time'
/usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/gems/activerecord-5.0.6/lib/active_record/migration.rb:815:in `say_with_time'
/usr/local/rvm/gems/ruby-2.4.0@katello-pr-test-29/gems/activerecord-5.0.6/lib/active_record/migration.rb:835:in `method_missing'
/var/lib/workspace/workspace/katello-pr-test/db/migrate/20171010172724_add_docker_manifest_list.rb:23:in `up'


··· On Wed, Nov 29, 2017 at 3:36 PM, Eric D Helms <ericdhelms@gmail.com> wrote:

Katello developers may notice a 'rails-5' status pop up on pull requests.
I am working to add testing against Rails 5 stack. This currently fails so
it can safely be ignored. This thread will get an update when the testing
has been officially implemented and thus should be a gate for new PRs.

--
Eric D. Helms
Red Hat Engineering



--
Eric D. Helms
Red Hat Engineering