| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-23 | |||
| 16:11:51 | gibi | but no luck reproducing the issue | |
| 16:12:07 | gibi | does it happen the same way for you each time you rerun these tests? | |
| 16:12:15 | gibi | or is it unstable? | |
| 16:13:47 | outbrito | consistent | |
| 16:15:30 | gibi | outbrito: do you also get a stack trace? If yes could you past it via https://paste.opendev.org/ (or similar) | |
| 16:17:04 | opendevreview | Merged openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 16:17:34 | outbrito | https://paste.opendev.org/show/bZm6XstZ4MTPM7SPVuCR/ | |
| 16:21:38 | gibi | thanks | |
| 16:25:06 | gibi | outbrito: what is the os-brick version in your virtualenv? | |
| 16:40:24 | mfo | sean-k-mooney, hey o/ getting back to you exactly 1 month later :) sorry. i'd appreciate your comments/review when you have a chance (but it's not urgent, as you can see). | |
| 16:40:35 | mfo | we chatted [0] about an issue [1] that the libvirt driver in uefi mode ended up with the ovmf _secboot_ image, which doesn't boot on the pc machine type (required q35); so i proposed to skip the secboot image if on pc. [2,3] | |
| 16:40:45 | mfo | [0] https://pastebin.ubuntu.com/p/jVxcXvDW6h/ | |
| 16:40:45 | mfo | [1] https://bugs.launchpad.net/nova/+bug/1960758 | |
| 16:40:45 | mfo | [2] https://review.opendev.org/c/openstack/nova/+/828980 | |
| 16:40:46 | mfo | [3] https://review.opendev.org/c/openstack/nova/+/828979 | |
| 16:40:53 | mfo | essentially, i would get back to you with: | |
| 16:40:53 | mfo | 1) explain why the secboot image didn't "just work", 2) add unit tests, 3) ensure we only skip secboot image if there's another image to use. | |
| 16:40:53 | mfo | i've explained 1) with technical details/research in the bug's comments, and addressed 2) and 3) in the patches. | |
| 16:40:55 | mfo | thanks! | |
| 16:41:53 | sean-k-mooney | mfo: ack, in a downstream call currently so will review it after | |
| 16:42:19 | sean-k-mooney | mfo: in your opipion are the parces ready for review if so ill try and add them to my backlog for this week | |
| 16:42:46 | mfo | sean-k-mooney, yup, i think it's ready for review. | |
| 16:43:49 | gibi | outbrito: sorry I don't see what is happening on your side with those test and I will soon and my day. | |
| 16:59:33 | outbrito | no worries, i'll keep investigating. Thanks for looking into it | |
| 17:00:38 | outbrito | gibi, os-brick is at 5.2.0 btw | |
| 18:44:31 | melwitt | dansmith: I have a patch proposed to address a func test gate bug I see semi frequently and the patch is based on a NOTE by you in the CellDatabases func test fixture. if you might be interested in reviewing: https://review.opendev.org/c/openstack/nova/+/831084 | |
| 18:53:26 | dansmith | melwitt: yeah I remember that, looking | |
| 19:02:48 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add whitebox-devstack-multinode job to periodic https://review.opendev.org/c/openstack/nova/+/833453 | |
| 19:04:49 | dansmith | melwitt: commented, see what you think | |
| 19:05:11 | melwitt | cool thanks. looking | |
| 19:09:22 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add whitebox-devstack-multinode job to periodic https://review.opendev.org/c/openstack/nova/+/833453 | |
| 19:16:27 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add whitebox-devstack-multinode job to periodic https://review.opendev.org/c/openstack/nova/+/833453 | |
| 19:37:30 | opendevreview | Artom Lifshitz proposed openstack/nova master: Add whitebox-devstack-multinode job to periodic https://review.opendev.org/c/openstack/nova/+/833453 | |
| 19:58:31 | artom | "deprecated_since=versionutils.deprecated.WALLABY" | |
| 19:58:43 | artom | ... in a placement installed in a ussuri tox functional venv | |
| 19:58:48 | artom | *sigh* | |
| 20:00:43 | sean-k-mooney | that is likely down to how we install placment for the placment fixture in the tox env | |
| 20:01:15 | sean-k-mooney | if that is happenign in a clean checkout with new venv | |
| 20:01:28 | sean-k-mooney | https://github.com/openstack/nova/blob/master/tox.ini#L119 | |
| 20:01:53 | sean-k-mooney | that should still be limited by upper constratis for the branch https://github.com/openstack/nova/blob/master/tox.ini#L29 | |
| 20:02:22 | sean-k-mooney | ah hehe | |
| 20:02:26 | sean-k-mooney | https://opendev.org/openstack/requirements/raw/branch/stable/ussuri/upper-constraints.txt | |
| 20:02:39 | sean-k-mooney | placement is not in upperconstrait for stable/ussuri | |
| 20:02:53 | sean-k-mooney | artom: so you will get whatever is on pypi | |
| 20:03:14 | sean-k-mooney | for the most part that should be ok as we are just using it for the fixture in this case | |
| 20:03:35 | artom | Unless it explicitly uses variables named after major releases :P | |
| 20:03:35 | sean-k-mooney | but that is why that happens its installing the latest pypi release of placement | |
| 20:03:56 | artom | Presumably this would fail in the upstream gate as well | |
| 20:04:15 | sean-k-mooney | proably we likely should cap it to the feature level | |
| 20:04:26 | sean-k-mooney | * minor release | |
| 20:04:47 | sean-k-mooney | actully placement is not release independ ot its the major verion | |
| 20:05:13 | sean-k-mooney | that woudl be pretty trival to do just update https://github.com/openstack/nova/blob/stable/ussuri/tox.ini#L91 | |
| 20:05:28 | sean-k-mooney | but we woudl need to do that with a stable only patch for each release | |
| 20:06:19 | sean-k-mooney | so ussuri is 3.0.0 so you need to set it to <4.0.0 | |
| 20:06:58 | sean-k-mooney | so """openstack-placement>=1.0.0,<4.0.0""" i think although im not sure if the , is needed | |
| 20:07:02 | sean-k-mooney | that shoudl fix it | |
| 20:07:34 | sean-k-mooney | or you coudl be explcit and say openstack-placement==3 | |
| 20:08:22 | sean-k-mooney | that will give you any 3.x.y but not 2.* or 4.* | |
| 20:08:47 | sean-k-mooney | artom: hopefully that makes sense | |
| 20:09:03 | sean-k-mooney | im goign to call it a day so chat to you tomorrow o/ | |
| 20:09:32 | artom | sean-k-mooney, yep, many thanks, very useful | |
| 20:10:00 | artom | Will also need to do it for victoria as well, since it doesn't understand WALLABY either | |
| 20:15:02 | clarkb | note I think ==3 means 3.0.0 not 3.x.y | |
| 20:16:14 | sean-k-mooney | im not sure about that but either way we can use min/max version or just pin to the majory with ==3 depending on the semantics of the syntax | |
| 20:16:30 | sean-k-mooney | i tought it woudl select the new 3.x.y | |
| 20:16:36 | sean-k-mooney | but im not sure | |
| 20:19:25 | clarkb | the ~= operator is for that or using * in the versions. missing identifiers seem to expand to .0 | |
| 20:19:59 | sean-k-mooney | ack | |
| 20:20:08 | sean-k-mooney | is ~= supprot new | |
| 20:20:33 | sean-k-mooney | im familar with it in other context but i did nto think it was supproted in the requirements specificaitons | |
| 20:21:59 | clarkb | it was defined in pep 440 somewhat new | |
| 20:28:27 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: [stable-only] Cap placement version in tox functional venv https://review.opendev.org/c/openstack/nova/+/834970 | |
| 23:37:15 | opendevreview | melanie witt proposed openstack/nova master: image_meta: Add ephemeral encryption properties https://review.opendev.org/c/openstack/nova/+/760454 | |
| 23:37:16 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | |
| 23:37:16 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | |
| 23:37:17 | opendevreview | melanie witt proposed openstack/nova master: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | |
| 23:37:17 | opendevreview | melanie witt proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| #openstack-nova - 2022-03-24 | |||
| 04:54:01 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add encryption fields https://review.opendev.org/c/openstack/nova/+/760453 | |
| 04:54:02 | opendevreview | melanie witt proposed openstack/nova master: BlockDeviceMapping: Add is_local property https://review.opendev.org/c/openstack/nova/+/764485 | |
| 04:54:02 | opendevreview | melanie witt proposed openstack/nova master: compute: Update bdms with ephemeral encryption details when requested https://review.opendev.org/c/openstack/nova/+/764486 | |
| 04:54:03 | opendevreview | melanie witt proposed openstack/nova master: virt: Add ephemeral encryption flag https://review.opendev.org/c/openstack/nova/+/760455 | |
| 15:54:57 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace implicit conversion of SELECT into FROM https://review.opendev.org/c/openstack/placement/+/800910 | |
| 15:54:58 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace 'as_scalar()' with 'scalar_subquery()' https://review.opendev.org/c/openstack/placement/+/801100 | |
| 15:54:58 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 15:54:59 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 15:54:59 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 15:55:00 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 15:55:00 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 15:55:01 | opendevreview | Stephen Finucane proposed openstack/placement master: tests: Restore - don't reset - warning filters https://review.opendev.org/c/openstack/placement/+/828119 | |
| 15:55:01 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Use Row, not LegacyRow https://review.opendev.org/c/openstack/placement/+/828305 | |
| 15:55:03 | opendevreview | Stephen Finucane proposed openstack/placement master: tox: Enable SQLAlchemy 2.0 warnings https://review.opendev.org/c/openstack/placement/+/801108 | |
| 18:46:07 | opendevreview | melanie witt proposed openstack/nova master: Retry in CellDatabases fixture when global DB state changes https://review.opendev.org/c/openstack/nova/+/831084 | |
| 19:33:08 | outbrito | gibi, g'day! Just FTR, I found out that just doing this I would get these tests running: https://paste.opendev.org/show/b7EqvAMhZE6fJe2bYNSI/ | |
| 19:33:29 | outbrito | Still investigating what could be different on my env | |
| 19:46:16 | outbrito | I'm almost sure this is because I'm running ubuntu desktop and I don't have iscsiadm on it | |
| 19:46:16 | outbrito | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/utils.py#L40 | |
| 20:09:47 | outbrito | Yeah, if I install open-iscsi the error ceases. Should that be on bindep then? | |
| 20:14:16 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 20:14:16 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 20:14:17 | opendevreview | Stephen Finucane proposed openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |