Skip to content

Add timeout knobs#25

Merged
kakkoyun merged 3 commits intoobservatorium:masterfrom
kakkoyun:timeouts
Apr 15, 2020
Merged

Add timeout knobs#25
kakkoyun merged 3 commits intoobservatorium:masterfrom
kakkoyun:timeouts

Conversation

@kakkoyun
Copy link
Copy Markdown
Member

@kakkoyun kakkoyun commented Apr 3, 2020

This adds all the possible knobs to be more flexible for the adjustments.
Also changes the default values.

@kakkoyun kakkoyun requested review from metalmatze and squat April 3, 2020 12:56
@metalmatze
Copy link
Copy Markdown
Contributor

Do we really want to expose all of these via flags? 🤔
Sane defaults should be good enough for 90% of these, don't you think?

@kakkoyun
Copy link
Copy Markdown
Member Author

kakkoyun commented Apr 3, 2020

@metalmatze Yes, happy to remove flags 👍 We have too many of them.

Comment thread internal/proxy/proxy.go Outdated
Comment thread internal/proxy/proxy.go Outdated
@kakkoyun kakkoyun force-pushed the timeouts branch 2 times, most recently from f520d7f to 5fc0a73 Compare April 15, 2020 08:57
Comment thread internal/proxy/proxy.go Outdated
Comment thread internal/server/server.go Outdated
Copy link
Copy Markdown
Member

@squat squat left a comment

Choose a reason for hiding this comment

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

One last set of nits for the comments and then let's merge :)

Copy link
Copy Markdown
Member

@squat squat left a comment

Choose a reason for hiding this comment

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

🚀

@squat
Copy link
Copy Markdown
Member

squat commented Apr 15, 2020

let's squash the commit :)

@kakkoyun kakkoyun merged commit e3ae04f into observatorium:master Apr 15, 2020
@kakkoyun kakkoyun deleted the timeouts branch April 15, 2020 09:30
periklis pushed a commit to periklis/api that referenced this pull request Jul 9, 2021
philipgough pushed a commit to philipgough/api that referenced this pull request Feb 1, 2024
* handle the path for different write backend
Signed-off-by: Marco llan@redhat.com

* fix test error
Signed-off-by: Marco llan@redhat.com

* fix test error
Signed-off-by: Marco llan@redhat.com
philipgough pushed a commit to philipgough/api that referenced this pull request Feb 10, 2026
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

Successfully merging this pull request may close these issues.

3 participants