Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConfigurationWorkflow only exists in under AnsibleTower::AutomationMa… #112

Merged
merged 1 commit into from
Jul 31, 2018
Merged

ConfigurationWorkflow only exists in under AnsibleTower::AutomationMa… #112

merged 1 commit into from
Jul 31, 2018

Conversation

jameswnl
Copy link
Contributor

@@ -27,6 +27,8 @@ class ManageIQ::Providers::AnsibleTower::AutomationManager < ManageIQ::Providers
require_nested :Refresher
require_nested :RefreshWorker

has_many :configuration_workflows, :dependent => :destroy, :foreign_key => "manager_id", :inverse_of => :manager
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ladas see this as well
This is due to the removal of the relationship from parent class https://github.com/ManageIQ/manageiq/pull/17720/files

@miq-bot
Copy link
Member

miq-bot commented Jul 31, 2018

Checked commit https://github.com/jameswnl/manageiq-providers-ansible_tower/commit/d125dd36ee4e139fda5bdceea18eaedccbaf4c02 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@Fryguy
Copy link
Member

Fryguy commented Jul 31, 2018

Merged core change.

@Fryguy Fryguy merged commit 52c99da into ManageIQ:master Jul 31, 2018
@Fryguy Fryguy modified the milestone: Sprint 91 Ending Jul 30, 2018 Jul 31, 2018
@Fryguy Fryguy modified the milestones: Sprint 91 Ending Jul 30, 2018, Sprint 92 Ending Aug 13, 2018 Jul 31, 2018
@Fryguy Fryguy self-assigned this Jul 31, 2018
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.

4 participants