gocd / kubernetes-elastic-agents

Kubernetes Elastic agent plugin for GoCD
https://www.gocd.org
Apache License 2.0
34 stars 32 forks source link

adding cluster timeout config #377

Closed artushin closed 10 months ago

artushin commented 10 months ago

This implements a configuration for setting timeouts for requests to the k8s cluster. GKE has been consistently outside the default timeout threshold for us, so we need a way to set .withRequestTimeout(pluginSettings.getClusterRequestTimeout()) on the Kubernetes Client.

Screenshot 2023-11-28 at 11 32 40 PM

chadlwilson commented 10 months ago

Thanks for the PR - seems like a sensible change to me and relatively straightforward to validate. Will take a look soon.

chadlwilson commented 10 months ago

Is it possible for you to allow changes to the PR from collaborators? Would like to make minor tweaks that are easier to "just make" than PR back-and-forth :-)

artushin commented 10 months ago

Is it possible for you to allow changes to the PR from collaborators? Would like to make minor tweaks that are easier to "just make" than PR back-and-forth :-)

For sure! Thanks for taking a look so quickly. I added you as a collaborator for the forked repo.

chadlwilson commented 10 months ago

Ahh, sorry, didn't mean full collaborator on the forked repo - that's a bit extreme. Normally on PRs there is a check-box to the right hand side which effectively grants specific access to the branch associated with the PR while the PR is open. This one:

image

I think it's not ticked in this case?

artushin commented 10 months ago

@chadlwilson It's weird, I'm just not seeing that option on this PR. I totally don't mind that if you just accept the invite to collaborate on the repo. If we do end up making further changes, we'll be putting them back in your hands for PR anyway :)

chadlwilson commented 10 months ago

Ahh no worries, I'll merge and tweak on master right prior to release, since there's no major blocker here.

artushin commented 10 months ago

Thanks @chadlwilson !

chadlwilson commented 9 months ago

Apologies for the delay, finally got around to sanity testing and releasing this in https://github.com/gocd/kubernetes-elastic-agents/releases/tag/v3.9.1-501