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

Add --wait, -w flag to run-task #2359

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Conversation

stephanme
Copy link
Contributor

When the wait flag is supplied to run-task, the cli will not exit until the task enters a terminal state; SUCCEEDED or FAILED.

The cli exits 0 when the task succeeds and exits 1 when the task fails.

This improves the scriptability of tasks. They can be run serially in a script.

Cherry-pick of 8671a5b (#1861) to v8
Addresses Issue #1104, #2238

Authored-by: Zach Robinson zrobinson@pivotal.io
Co-authored-by: Stephan Merker stephan.merker@sap.com

Does this PR modify CLI v6, CLI v7, or CLI v8?

CLI v8

Description of the Change

Adds --wait flag to run-task command as available in CLI v7.

Why Is This PR Valuable?

This improves the scriptability of tasks. They can be run serially in a script.
The --wait flag is supported for the run-task command in v7.

Why Should This Be In Core?

New cmd line flag for existing core command.

Applicable Issues

#2238

Original issue for v7 was #1104

How Urgent Is The Change?

not urgent

Other Relevant Parties

n/a

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 8, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: stephanme / name: Stephan Merker (f3a39d4)

Copy link
Member

@a-b a-b left a comment

Choose a reason for hiding this comment

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

LGTM

@a-b
Copy link
Member

a-b commented Dec 13, 2022

Hi @stephanme, can you please fix EasyCLA? So we can proceed with merging this PR.

@jdgonzaleza
Copy link
Contributor

LGTM

@remsikr
Copy link

remsikr commented Sep 27, 2023

Hello,
when --wait, -w flag to run-task will be available in v8?

When the wait flag is supplied to run-task, the cli will
not exit until the task enters a terminal state; SUCCEEDED or FAILED.

The cli exits 0 when the task succeeds and exits 1 when the task fails.

This improves the scriptability of tasks. They can be run
serially in a script.

Cherry-pick of 8671a5b (cloudfoundry#1861) to v8
Addresses Issue cloudfoundry#1104, cloudfoundry#2238

Authored-by: Zach Robinson <zrobinson@pivotal.io>
Co-authored-by: Stephan Merker <stephan.merker@sap.com>
@ccjaimes ccjaimes merged commit 8224c92 into cloudfoundry:v8 Oct 12, 2023
ccjaimes pushed a commit that referenced this pull request Oct 12, 2023
When the wait flag is supplied to run-task, the cli will
not exit until the task enters a terminal state; SUCCEEDED or FAILED.

The cli exits 0 when the task succeeds and exits 1 when the task fails.

This improves the scriptability of tasks. They can be run
serially in a script.

Cherry-pick of 8671a5b (#1861) to v8
Addresses Issue #1104, #2238

Authored-by: Zach Robinson <zrobinson@pivotal.io>
ccjaimes added a commit that referenced this pull request Oct 13, 2023
When the wait flag is supplied to run-task, the cli will
not exit until the task enters a terminal state; SUCCEEDED or FAILED.

The cli exits 0 when the task succeeds and exits 1 when the task fails.

This improves the scriptability of tasks. They can be run
serially in a script.

Cherry-pick of 8671a5b (#1861) to v8
Addresses Issue #1104, #2238

Authored-by: Zach Robinson <zrobinson@pivotal.io>

Co-authored-by: Stephan Merker <merker.stephan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants