Skip to content
This repository has been archived by the owner on Sep 15, 2021. It is now read-only.

Ordering: added Target and Source to Task, removed waiting #84

Conversation

slemrmartin
Copy link
Contributor

@slemrmartin slemrmartin commented Apr 14, 2020

Issue #83

Operation now just orders ServiceOffering and doesn't wait 30 minutes to completing. This functionality is moved to topological_inventory-core.


def wait_for_job_finished(task_id, job, context)
count = 0
last_status = nil
timeout_count = POLL_TIMEOUT / SLEEP_POLL
Copy link
Contributor

Choose a reason for hiding this comment

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

We can remove POLL_TIMEOUT & SLEEP_POLL correct?

@syncrou
Copy link
Contributor

syncrou commented Apr 20, 2020

@slemrmartin Looks like there is a conflict as well.

@slemrmartin slemrmartin force-pushed the operations-removed-waiting-for-order-complete branch from c1f1a2f to e6e2e0f Compare April 22, 2020 08:29
@agrare agrare self-assigned this Apr 22, 2020
@agrare agrare closed this Apr 22, 2020
@agrare agrare reopened this Apr 22, 2020
@agrare agrare merged commit 8adc592 into RedHatInsights:master Apr 22, 2020
@slemrmartin slemrmartin deleted the operations-removed-waiting-for-order-complete branch April 22, 2020 12:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants