*** akahat|out is now known as akahat|ruck | 05:15 | |
*** jpena|off is now known as jpena | 07:39 | |
opendevreview | Slawek Kaplonski proposed openstack/devstack master: Fix missing "$" in the ENFORCE_SCOPE's variable name https://review.opendev.org/c/openstack/devstack/+/848083 | 07:58 |
---|---|---|
opendevreview | Martin Kopec proposed openstack/devstack master: Make devstack-platform-debian-bullseye voting https://review.opendev.org/c/openstack/devstack/+/848086 | 08:44 |
opendevreview | Merged openstack/tempest master: Use yoga stable contraint in tox to release 31.1.0 https://review.opendev.org/c/openstack/tempest/+/847849 | 09:42 |
opendevreview | Merged openstack/tempest master: Switch the tox contraints back to master https://review.opendev.org/c/openstack/tempest/+/847850 | 09:42 |
opendevreview | Martin Kopec proposed openstack/tempest master: Add release notes page for version 31.1.0 https://review.opendev.org/c/openstack/tempest/+/847851 | 09:45 |
opendevreview | Martin Kopec proposed openstack/tempest master: DNM Test tempest with enforce scope https://review.opendev.org/c/openstack/tempest/+/847760 | 11:17 |
*** pojadhav is now known as pojadhav|afk | 12:52 | |
*** pojadhav|afk is now known as pojadhav | 14:35 | |
opendevreview | Merged openstack/devstack master: Fix missing "$" in the ENFORCE_SCOPE's variable name https://review.opendev.org/c/openstack/devstack/+/848083 | 16:05 |
*** jpena is now known as jpena|off | 16:34 | |
tosky | gmann, kopecmartin: hi, could you please check this devstack change? https://review.opendev.org/c/openstack/devstack/+/834269/ It is needed by https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/826484/ | 16:46 |
gmann | tosky: yeah on it | 16:47 |
tosky | thanks | 16:48 |
gmann | tosky: thanks for pointing out devstack-plugin-ceph patch, I was searching in master code of it about cephadm | 16:53 |
gmann | +A | 16:53 |
tosky | I'm quite sure the message returned by gerrit in such cases was different in the past, but maybe I haven't noticed such cases before | 17:00 |
*** lbragstad1 is now known as lbragstad | 17:57 | |
opendevreview | Merged openstack/devstack master: Do not create cinder backup pool and key when cephadm is used https://review.opendev.org/c/openstack/devstack/+/834269 | 19:02 |
opendevreview | Ilya Popov proposed openstack/tempest master: Fix error KeyError: 'addresses' in get_server_ip https://review.opendev.org/c/openstack/tempest/+/841719 | 19:18 |
melwitt | gmann: I tried the required-projects and got "This change depends on a change that failed to merge." https://review.opendev.org/c/openstack/nova/+/843700 do you think that's related to required-projects or do you think I need to rebase? | 22:11 |
clarkb | melwitt: if i had to guess https://review.opendev.org/c/openstack/tempest/+/836065 may need a rebase | 22:14 |
clarkb | since zuul specifically says it is a dependency that fails to merge | 22:14 |
melwitt | clarkb: thanks for that, I will check it | 22:15 |
gmann | clarkb: melwitt: not sure about that, it should work as there is no change in depends-on and they are up to date? or tempest patch is in conflict ? | 22:41 |
gmann | clarkb: ohk. I think that is the case | 22:42 |
opendevreview | Ghanshyam proposed openstack/tempest master: Add LUKS v2 tests https://review.opendev.org/c/openstack/tempest/+/836065 | 22:42 |
gmann | melwitt: clarkb ^^ seems working now | 22:43 |
gmann | strange as there was no conflict in tempest patch with master | 22:44 |
melwitt | gmann: thanks! and +1, I don't understand it either given no conflict when I did a local rebase | 22:44 |
gmann | yeah, that is confusing for me too | 22:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!