| *** ysandeep|out is now known as ysandeep | 05:27 | |
| opendevreview | OpenStack Proposal Bot proposed openstack/heat master: Imported Translations from Zanata https://review.opendev.org/c/openstack/heat/+/809949 | 06:42 |
|---|---|---|
| *** ysandeep is now known as ysandeep|lunch | 08:28 | |
| zigo | Hi there! We're trying to fix a long standing issue, where we have the Heat API endpoint setup at something like https://example.com/orchestration-api/v1/%(tenant_id)s and heat is builing links in its stack forgetting the "/orchestration-api" part of the URL. We're struggling to find out where the error is in the code. Would someone know where we should look into? | 09:36 |
| zigo | zaneb: Any clue? Would that be in make_url() ? | 09:36 |
| *** ysandeep|lunch is now known as ysandeep | 09:46 | |
| *** hjensas is now known as hjensas|lunch | 09:58 | |
| *** ysandeep is now known as ysandeep|away | 11:00 | |
| opendevreview | Przemyslaw Szczerbik proposed openstack/heat master: Add OS::Neutron::QoSMinimumBandwidthRule resource https://review.opendev.org/c/openstack/heat/+/812662 | 11:02 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/heat master: Add OS::Neutron::QoSMinimumPacketRateRule resource https://review.opendev.org/c/openstack/heat/+/812662 | 11:53 |
| *** ysandeep|away is now known as ysandeep | 12:44 | |
| zaneb | zigo: maybe? apparently I wrote that code... 9 years ago :D | 13:31 |
| zigo | :) | 13:32 |
| zaneb | where exactly is the bad link showing up? | 13:32 |
| zigo | zaneb: I lost track of who's doing what, since we can't gather as real persons ... | 13:32 |
| zigo | Hang on, asking my colleague for it. | 13:33 |
| zaneb | given that all URLs in the routing setup start with /, it seems entirely possible that when a request comes in the part of the path up to the endpoint has already been stripped, so that calling req.relative_url() would give a path with everything before /v1 missing | 13:41 |
| zigo | zaneb: https://paste.openstack.org/show/809819/ <--- If you see here, the href should be https://preprod-api.cloud.infomaniak.ch/orchestration-api/v1/df6d... | 13:49 |
| zigo | Somehow, Heat believes everyone uses the root of the server, when we're using /orchestration-api/v1 instead ... | 13:50 |
| zaneb | zigo: thanks, yeah it looks like make_url() would be the culprit | 13:50 |
| zigo | zaneb: Allright, thanks. | 13:59 |
| zigo | What surprised me though, is that devstack is setting-up endpoints the same way we do (ie: not at the root), no ? | 13:59 |
| zigo | So it should work, no ? | 13:59 |
| zigo | Or is there somewhere to configure that thing so it would write things correctly ? | 13:59 |
| *** ysandeep is now known as ysandeep|out | 16:15 | |
| zaneb | zigo: check that the SCRIPT_NAME environment variable gets set to "/orchestration-api" by whatever environment is hosting heat-api (uwsgi, apache) | 17:03 |
| zaneb | the code looks correct in principle | 17:04 |
| *** ysandeep|out is now known as ysandeep | 18:03 | |
| *** ysandeep is now known as ysandeep|out | 19:24 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!