*** mhen_ is now known as mhen | 01:17 | |
opendevreview | Ghanshyam proposed openstack/tempest master: Test project manager role for server migration https://review.opendev.org/c/openstack/tempest/+/953265 | 03:32 |
---|---|---|
*** elodilles is now known as elodilles_pto | 06:14 | |
opendevreview | Merged openstack/grenade master: Zuul: do not use USE_PYTHON3 https://review.opendev.org/c/openstack/grenade/+/953495 | 07:27 |
opendevreview | Stephen Finucane proposed openstack/devstack master: Fail on failed db sync https://review.opendev.org/c/openstack/devstack/+/953778 | 14:23 |
stephenfin | ralonsoh: don't know if that will work or not but it's a start ^ | 14:23 |
gboutry | Hey, looking into an issue I'm getting with python-tempestconf where basically, I have the user operation that script in a different domain that the user in identity.username, but it looks like the domain support isn't entirely there or it doesn't behave as I expect. | 14:27 |
gboutry | I would expect identity.user_domaine_name to define under which domain the user is created, but it seems to be ignored and the user is created in the default domain. | 14:27 |
gboutry | Is that known? Is there any other way to create a user in a specific domain with python-tempestconf? (other than creating the user myself beforehand) | 14:27 |
gboutry | If not, I started working on a patch, but would rather discuss it before going too far. | 14:27 |
ralonsoh | stephenfin, let me check it | 14:29 |
gmaan | gboutry: python-tempestconf was maintained by the interop group which was closed officially but this is IRC channel if you get someone reply there #openstack-interop | 15:36 |
tosky | gboutry: python-tempestconf core here; we are not very active, and any patch is welcome :) | 16:14 |
tosky | not all use cases are supported or implemented | 16:15 |
tosky | and there are some assumption on the environment where tempestconf is executed: some bits are handled by tempestconf, but not always | 16:15 |
gboutry | What are these assumptions? demo users being created in the default domain is one of these? | 16:18 |
opendevreview | Ghanshyam proposed openstack/tempest master: Test project manager role for server migration https://review.opendev.org/c/openstack/tempest/+/953265 | 16:21 |
tosky | gboutry: we probably never thought about making it configurable | 16:42 |
gboutry | tosky: thanks for the replies, do you think it would make sense to create the demo users in the same domain as identity.user_domaine_name, or this would need a new config option? | 16:51 |
gboutry | This would change existing behaviour of the tool. | 16:51 |
tosky | I would go for a change that doesn't change the default behavior, something that can be triggered as an option | 16:54 |
*** iurygregory__ is now known as iurygregory | 16:58 | |
* tosky bbl | 16:58 | |
*** iurygregory_ is now known as iurygregory | 18:07 | |
sean-k-mooney | fungi: so on the signed off by enforcement we exipclty ask to not require it for enw revions of exting patches to not need to add it to any open cherry picke that need a rebase. i assume the tooling cant differenciate and we jsut have to live with that as a sideefect | 19:53 |
sean-k-mooney | fungi: if this woudl need any more work then setting a boolean flag im oke with adding the signed off by retoactivly its just annoying | 19:54 |
fungi | sean-k-mooney: yes, it's checked at push, regardless of whether it's for an existing or new change | 19:54 |
sean-k-mooney | ack | 19:55 |
fungi | doesn't need to be added retroactively, only when pushing a revision | 19:55 |
fungi | existing changes that aren't being revised don't need it | 19:55 |
fungi | they can merge as-is | 19:55 |
sean-k-mooney | right i just thinkign of when we need to rebase pendign backprots | 19:55 |
sean-k-mooney | its fine | 19:55 |
sean-k-mooney | we have a couple of large serise in flight but i think at least stephen ahs already been usign it for there openapi series | 19:56 |
sean-k-mooney | gibi has not for there eventlet removal serise in nova | 19:57 |
sean-k-mooney | we knew this was comming so it is what it is | 19:58 |
sean-k-mooney | it would have been nice ot only need ti for new reviews | 19:58 |
fungi | at least it emits a clear error message, so should be obvious what needs doing whenever you trip over it | 20:04 |
opendevreview | Ghanshyam proposed openstack/tempest master: Add more server migration tests https://review.opendev.org/c/openstack/tempest/+/953847 | 22:35 |
opendevreview | Ghanshyam proposed openstack/tempest master: Add more server migration tests https://review.opendev.org/c/openstack/tempest/+/953847 | 22:39 |
opendevreview | Ghanshyam proposed openstack/tempest master: Move migration scenario tests to use project manager https://review.opendev.org/c/openstack/tempest/+/953850 | 23:40 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!