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

Quiet period calculator cannot be changed #200

Closed
dmartell-bp opened this issue Apr 12, 2020 · 12 comments
Closed

Quiet period calculator cannot be changed #200

dmartell-bp opened this issue Apr 12, 2020 · 12 comments

Comments

@dmartell-bp
Copy link

The Quiet period calculator has a default of "index < 5 ? 0 : 2 * 60" but cannot be changed. This is a major bug for anyone trying to spin up more than 4 simultaneous test environments using the MultiJob plugin.

@dmartell-bp dmartell-bp changed the title Quiet period caluclator cannot be changes Quiet period caluclator cannot be changed Apr 12, 2020
@dmartell-bp dmartell-bp changed the title Quiet period caluclator cannot be changed Quiet period calculator cannot be changed Apr 12, 2020
@caseyduquettesc
Copy link

Luckily, looking at the code, this looks more like a UI bug. You can confirm if you look at your job config history, it saves the values you enter, but when edit the job, it always loads the default value. This unfortunately means you will need to be careful whenever you save the job to always enter a sane value.

Personally, I'll probably go back to 1.32.

@xUrko
Copy link

xUrko commented May 5, 2020

I am still running 1.31 since 1.32 also did not work properly.

@RochaStratovan
Copy link

Why is the default value anything other than 0. We've had sizable performance hits due to this default value being applied because of some trivial unrelated edit to the job description. It may be caused by a UI issue, but it's a rather serious issue since it silently changes unrelated values.

@flocsy
Copy link

flocsy commented May 21, 2020

I agree, please set the default value to be 0, that's backwards compatible with how it behaved before the quietPeriodCalculator was added.

@RochaStratovan
Copy link

Any idea on when this critical defect will be fixed?

@srilakalways
Copy link

I agree, please set the default value to be 0, that's backwards compatible with how it behaved before the quietPeriodCalculator was added.

Where can the value be changed globally?

@anoymouserver
Copy link

should be fixed with #201

@RochaStratovan
Copy link

Do we know when this will be released? I have latest 1.33 and I still see this issue.

@ccnet-microarea
Copy link

ccnet-microarea commented Oct 8, 2020

Do we know when this will be released? I have latest 1.33 and I still see this issue.

Me too! That's the reason because I had to downgrade to 1.32, but 1.32 has the Next phase is EXECUTING even though the previous phase is FAILED issue, which is pretty critical for us...

@RochaStratovan
Copy link

Do we know when this will be released? I have 1.33 and I still see this issue. This is a configuration corruption. A value unrelated to what we are editing is being changed, and it causes problems.

@naqushab
Copy link

Any update on the fix? Is there a workaround to this?

@yorammi
Copy link
Contributor

yorammi commented Dec 29, 2020

We released 1.36 (at last - we have odd authentication issues with jenkins-CI) and it includes a fix for this issue.

@yorammi yorammi closed this as completed Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants