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

[FEATURE] Set default schedule to "paused" for any Job in Test Env #75

Open
1 of 4 tasks
jcnorman48 opened this issue Jan 2, 2024 · 4 comments
Open
1 of 4 tasks
Assignees
Labels
enhancement New feature or request

Comments

@jcnorman48
Copy link

Is your feature request related to a problem? Please describe.
In order to minimize cost, any deployed job that is running on a schedule should have a state set as paused in the test environment.

Cloud Information

  • AWS
  • Azure
  • GCP
  • Other

Describe the solution you'd like
Any deployment of a Job/pipelines in brickflow should be set as "paused" if specified to be running on a schedule when deployed to any non prod environment.

Describe alternatives you've considered

Additional context

@jcnorman48 jcnorman48 added the enhancement New feature or request label Jan 2, 2024
@asingamaneni
Copy link
Collaborator

We also need to probably need to hard set the schedule as None if the env is local.

@asingamaneni
Copy link
Collaborator

For local: we will default it to None, but will give an option to override and schedule it
For dev, qa & prod: Folks should be able to decide the schedule in the workflow using ctx.env and conditional statements

@asingamaneni asingamaneni self-assigned this Feb 1, 2024
@pariksheet
Copy link
Collaborator

For local: we will default it to None, but will give an option to override and schedule it For dev, qa & prod: Folks should be able to decide the schedule in the workflow using ctx.env and conditional statements

for local -- instead of setting schedule to None, we should make the workflow status PAUSE.

There are time, when team wants test Quartz expressions. PAUSED is better than override flag.

@asingamaneni
Copy link
Collaborator

I agree, makes sense!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants