-
Notifications
You must be signed in to change notification settings - Fork 3
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
Introducing new lane to test PROXY now and later Rancher upgrade testing #206
base: main
Are you sure you want to change the base?
Conversation
refresh_pattern -i (/cgi-bin/|\?) 0 0% 0 | ||
refresh_pattern . 0 20% 4320 | ||
|
||
debug_options ALL,1 11,3 20,3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a newline
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a question. Other than that we will see how it goes once part 2 is on.
Approved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is incomplete and can't work as it is:
- missing proxy configuration for underlying k8s
- ele-testhelpers for setting rancher proxy URL is using hardcoded value which can be different in your environment
66a918c
to
8b7d03e
Compare
Currently, I am working on this to accommodate changes to existing
This is done. |
8b7d03e
to
17fb38c
Compare
Only missing piece is the k3s configuration :) What prevents you to add something like this into install_suite.go when proxy is ON?
|
Nothing is stopping me to add this option to K3s, I will add it and update the PR. |
5b50554
to
f3838f1
Compare
997772d
to
c588817
Compare
cdd4cca
to
6e40534
Compare
Signed-off-by: Satyajit Bulage <[email protected]>
Signed-off-by: Satyajit Bulage <[email protected]>
6e40534
to
56d66a8
Compare
Partly fixes: #196