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
Fixes #7362 - Update org create dynflow #4642
Conversation
0cfa346
to
0e0679c
Compare
@parthaa looks like this needs a rebase, I'll attempt to test it once you do |
@ehelms rebased |
self.organization.reload #must reload organization, otherwise old name is saved | ||
self.organization.update_index | ||
#self.organization.reload #must reload organization, otherwise old name is saved | ||
#self.organization.update_index |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove?
I attempted, through the UI to create an org with the following: Name: Bvlah And got this in the UI:
|
6c8ddc1
to
7116397
Compare
@ehelms try now |
[test] |
[test] |
1 similar comment
[test] |
|
||
sequence do | ||
if ::Katello.config.use_cp | ||
cp_create = plan_action(Candlepin::Owner::Create, | ||
label: organization.label, | ||
name: organization.name) | ||
|
||
plan_action(Environment::LibraryCreate, organization.library) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this action inside the Katello.config.use_cp
condition?
Moved it to follow org destroy model. Removed the create orchestration off sync_action and made the controllers use the orchestrated action directly. Needed this for tests related to issue Katello#6946 to work. Also updated unit tests to deal with this change. Also removed the unused org elastic search bindings more more
ACK |
Fixes #7362 - Update org create dynflow
Moved it to follow org destroy model. Removed the create orchestration off
sync_action and made the controllers use the orchestrated action
directly. Needed this for tests related to issue #6946 to work.
Also updated unit tests to deal with this change.