| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Add option to SKIP Actions https://review.opendev.org/c/openstack/watcher-dashboard/+/958209 | 05:47 |
|---|---|---|
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Fix translation context for watcher action states https://review.opendev.org/c/openstack/watcher-dashboard/+/959189 | 05:48 |
| opendevreview | chandan kumar proposed openstack/watcher-dashboard master: Show parameter spec in the strategy info page https://review.opendev.org/c/openstack/watcher-dashboard/+/960232 | 06:19 |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: Split execute_strategy into smaller functions https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/960258 | 12:23 |
| opendevreview | Joan Gilabert proposed openstack/watcher master: Handle missing fields building storage model XML https://review.opendev.org/c/openstack/watcher/+/960265 | 13:43 |
| tkajinam | sean-k-mooney, just fyi... I spent some time to look into the current situation of proxy header parsing in watcher and eventually learned that recent mod_wsgi, uwsgi and gunicorn handles in in their layer ... https://bugs.launchpad.net/watcher/+bug/2122353 | 16:01 |
| tkajinam | sean-k-mooney, so as long as we set the right headers in frontend loadbalancer python apps may be able to pick up the right application url without any middleware in the middle | 16:02 |
| tkajinam | this indicates that we can even get rid of the http_proxy_to_wsgi middleware. phhhh | 16:02 |
| sean-k-mooney | hum i see | 16:03 |
| sean-k-mooney | so i guess that is only needed if runing under the eventlet webserver then | 16:03 |
| tkajinam | yeah that's true | 16:03 |
| sean-k-mooney | i think we removed that this cycle or planned to do it next cycle | 16:04 |
| sean-k-mooney | so i think we can moretly ignore that | 16:04 |
| sean-k-mooney | in any case i think we shoudl relflect on our documtionation and testing in ci to ensure we are not missing anything relevent related to https://bugs.launchpad.net/watcher/+bug/2122353 | 16:05 |
| sean-k-mooney | so we could use this to track making sure we have teh tls_proxy enabled in our devstack jobs and ensure that the urls are corerct | 16:06 |
| tkajinam | I was able to reproduce it with standalone watcher-api (which uses eventlet) so I reopened the bug with that information... though we may eventually close it as won't fix when we get rid of eventlet server. | 16:11 |
| tkajinam | watcher-api console script is still there in master, iiuc | 16:11 |
| tkajinam | a bit annoying thing is that current watcher-tempest-plugin asserts values in href but it does not assert the protocol in these links strictly | 16:11 |
| tkajinam | I mean it checks only 'https?' regex regardless of actual protocol used | 16:12 |
| opendevreview | sean mooney proposed openstack/watcher master: Add 2025.2 release notes prelude https://review.opendev.org/c/openstack/watcher/+/960298 | 16:55 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Add capability to parse forward headers https://review.opendev.org/c/openstack/watcher/+/960157 | 17:39 |
| opendevreview | Takashi Kajinami proposed openstack/watcher master: Add capability to parse forward headers https://review.opendev.org/c/openstack/watcher/+/960157 | 17:39 |
| opendevreview | Douglas Viroel proposed openstack/watcher-tempest-plugin master: Refactor execute_strategy method into smaller helpers https://review.opendev.org/c/openstack/watcher-tempest-plugin/+/960310 | 19:43 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!