| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-04 | |||
| 18:46:33 | sean-k-mooney | i mean its ment to have supprot | |
| 18:46:54 | sean-k-mooney | i see stuff as far back as 4.7.0 | |
| 18:46:57 | sean-k-mooney | in the release notes | |
| 18:47:08 | chateaulav | it seems like partial support, gonna test along that route | |
| 18:47:50 | sean-k-mooney | maybe they dont support emulation with riscv | |
| 18:47:56 | sean-k-mooney | just native supprot? | |
| 18:49:47 | chateaulav | I feel that as well, a couple last checks and then ill put a nail in it | |
| 19:56:23 | opendevreview | Artom Lifshitz proposed openstack/nova master: DNM: Run Nova gate on Neutron notifier patch https://review.opendev.org/c/openstack/nova/+/827917 | |
| 19:59:09 | opendevreview | Ade Lee proposed openstack/nova stable/xena: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/827895 | |
| 20:00:39 | opendevreview | Ade Lee proposed openstack/nova stable/xena: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/827895 | |
| 20:01:28 | opendevreview | Ade Lee proposed openstack/nova stable/wallaby: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/827896 | |
| 20:03:10 | melwitt | dansmith: hi :D I have been working on removing tracebacks in the logs in nova-multi-cell that happen when a neutron network-changed event is received after the instance has moved to another cell during a cross-cell migration, | |
| 20:04:32 | melwitt | this seemed simple at first until the fact that the exception is being raised from the db access on nova-conductor but the tracing is happening in nova-compute, | |
| 20:05:24 | sean-k-mooney | it helps if you enable forwarding on the host if you want nat to work for your vms.... sudo sysctl net.ipv4.conf.all.forwarding=1 | |
| 20:05:44 | dansmith | melwitt: because of indirection I assume | |
| 20:06:08 | melwitt | I'm seeing tracing still from the rpc messaging layer, so I tried this https://review.opendev.org/c/openstack/nova/+/826942/5/nova/objects/instance_info_cache.py#77 but it doesn't work as expected. I get oslo_messaging.rpc.client.RemoteError: Remote error: ExpectedException on the client side instead of InstanceNotFound | |
| 20:06:58 | melwitt | I had thought when oslo.messaging receives ExpectedException on the client side it translates it into the original exception type that it wrapped on the server side, but that is not occurring | |
| 20:07:56 | dansmith | no, I think that just avoids the traceback on the conductor side | |
| 20:08:14 | melwitt | hm ok | |
| 20:08:21 | sean-k-mooney | melwitt: i assume you are not checkign if the instance exists then updating because it could get deleted in between | |
| 20:08:28 | sean-k-mooney | which is why you are trying to catch the excpiton | |
| 20:08:34 | melwitt | sean-k-mooney: yeah | |
| 20:09:06 | melwitt | well, I'm catching DBReferenceError and then raising InstanceNotFound to be handled by upper layers | |
| 20:09:49 | sean-k-mooney | do the upper layers need to do anything with the not found | |
| 20:09:57 | sean-k-mooney | the instance has been moved to the other cell yes | |
| 20:10:50 | melwitt | most things handle not found, like "the instance disappeared while <whatever>" | |
| 20:11:07 | melwitt | if someone deletes while it's in the middle of <whatever> | |
| 20:11:36 | melwitt | it's working fine except I'm still getting tracebacks in nova-compute coming from oslo.messaging and I can't get it to stop | |
| 20:11:49 | sean-k-mooney | ack | |
| 20:13:41 | melwitt | it sends over the longass chain of exception tracebacks even though I caught DBReferenceError on the conductor side | |
| 20:29:18 | dansmith | melwitt: I think oslo.messaging wraps exceptions in a special class called Original_Remote, IIRC | |
| 20:29:26 | dansmith | not sure you can catch that directly though because I think it's dynamic | |
| 20:29:44 | dansmith | if you catch it with except Exception, then try to print e.__class__ | |
| 20:33:01 | melwitt | dansmith: yeah, it's weird. it's as if it's logging it somewhere I can't get to (like in oslo.messaging). this is an example from PS3 before I added any @expected_exceptions, it logs at INFO what I wanted and then directly after that it logs the supertraceback https://zuul.opendev.org/t/openstack/build/7c3c755868b848ed841d37457004d99b/log/controller/logs/screen-n-cpu.txt#10027 | |
| 20:33:44 | melwitt | I don't think? it's just appended to the exception message bc it's logged in a different color but since I'm desperate I might try truncating str(e) to see what happens | |
| 20:36:15 | dansmith | yeah I expect it's logged explicitly somewhere, but not sure | |
| 20:36:21 | dansmith | nothing specific jumps out at me though | |
| 20:39:50 | melwitt | dansmith: ok, I'll try some more things. thanks for looking :) | |
| 21:01:58 | opendevreview | Jonathan Race proposed openstack/nova master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 21:28:05 | opendevreview | Ade Lee proposed openstack/nova stable/wallaby: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/827896 | |
| 22:09:19 | opendevreview | melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942 | |
| 22:12:01 | opendevreview | Ade Lee proposed openstack/nova stable/wallaby: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/827896 | |
| #openstack-nova - 2022-02-05 | |||
| 00:31:12 | opendevreview | Merged openstack/nova stable/wallaby: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/c/openstack/nova/+/811806 | |
| 00:36:46 | opendevreview | Merged openstack/nova master: skip test_tagged_attachment in nova-next https://review.opendev.org/c/openstack/nova/+/827851 | |
| 01:12:51 | opendevreview | melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942 | |
| 02:01:01 | opendevreview | melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942 | |
| 07:23:41 | opendevreview | melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942 | |
| 09:33:36 | opendevreview | Merged openstack/nova master: [yoga] Add PCI VPD Capability Handling https://review.opendev.org/c/openstack/nova/+/808199 | |
| 16:52:55 | opendevreview | Takashi Kajinami proposed openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 16:54:17 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Replace implicit conversion of SELECT into FROM https://review.opendev.org/c/openstack/placement/+/800910 | |
| 16:54:50 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Replace 'as_scalar()' with 'scalar_subquery()' https://review.opendev.org/c/openstack/placement/+/801100 | |
| 16:56:04 | opendevreview | Takashi Kajinami proposed openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 16:57:15 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 16:57:29 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 16:57:37 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 16:57:43 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 16:57:48 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 17:01:02 | opendevreview | Takashi Kajinami proposed openstack/placement master: tests: Silence noisy tests https://review.opendev.org/c/openstack/placement/+/801101 | |
| 17:01:02 | opendevreview | Takashi Kajinami proposed openstack/placement master: tox: Remove psycopg2 warning filter https://review.opendev.org/c/openstack/placement/+/800911 | |
| 17:01:03 | opendevreview | Takashi Kajinami proposed openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 17:01:03 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Replace implicit conversion of SELECT into FROM https://review.opendev.org/c/openstack/placement/+/800910 | |
| 17:01:04 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Replace 'as_scalar()' with 'scalar_subquery()' https://review.opendev.org/c/openstack/placement/+/801100 | |
| 17:01:04 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Update 'select()' calls https://review.opendev.org/c/openstack/placement/+/801103 | |
| 17:01:05 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Remove use of non-integer/slice indices https://review.opendev.org/c/openstack/placement/+/801104 | |
| 17:01:06 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Replace deprecated 'FromClause.select().whereclause' parameter https://review.opendev.org/c/openstack/placement/+/801105 | |
| 17:01:06 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Use explicit transactions https://review.opendev.org/c/openstack/placement/+/801106 | |
| 17:01:08 | opendevreview | Takashi Kajinami proposed openstack/placement master: db: Remove unnecessary use of '_mapping' https://review.opendev.org/c/openstack/placement/+/801107 | |
| 17:01:08 | opendevreview | Takashi Kajinami proposed openstack/placement master: tox: Enable SQLAlchemy 2.0 warnings https://review.opendev.org/c/openstack/placement/+/801108 | |
| 17:21:37 | opendevreview | Takashi Kajinami proposed openstack/os-resource-classes master: setup.cfg: Replace dashes by underscores https://review.opendev.org/c/openstack/os-resource-classes/+/828009 | |
| #openstack-nova - 2022-02-06 | |||
| 12:00:57 | opendevreview | Ilya Popov proposed openstack/nova master: Fix to implement 'pack' or 'spread' VM's NUMA cells https://review.opendev.org/c/openstack/nova/+/805649 | |
| #openstack-nova - 2022-02-07 | |||
| 07:20:39 | opendevreview | Attila Fazekas proposed openstack/nova master: Note the deleyad address view https://review.opendev.org/c/openstack/nova/+/827856 | |
| 08:34:30 | yuval | Hey is there any special keyword I can reply to my patch to make zuul re-test my patch? | |
| 08:36:45 | gibi | yuval: say: recheck <reason, e.g. the bug number your patch hit> | |
| 08:42:07 | yuval | gibi: in this chat or on the gerrit gui? | |
| 08:42:25 | gibi | yuval: in gerrit as a reply | |
| 08:42:30 | yuval | thanks | |
| 10:27:53 | opendevreview | Merged openstack/nova master: docs: Follow-ups for cells v2, architecture docs https://review.opendev.org/c/openstack/nova/+/827336 | |
| 11:25:36 | opendevreview | Tobias Urdin proposed openstack/nova master: Update announce self workaround opt description https://review.opendev.org/c/openstack/nova/+/826829 | |
| 11:34:38 | gibi | sean-k-mooney: hi! I'm +2 on the off-path networking series now | |
| 11:35:00 | sean-k-mooney | ack that is nice to hear | |
| 11:39:38 | gibi | sean-k-mooney: also looked at your healthcheck wip patch and left some notes | |
| 11:51:41 | sean-k-mooney | thanks i need to spend more time on that in general but once i have the basic infra working adding the checks should be fairly quick | |
| 12:00:37 | gibi | yepp | |
| 12:03:43 | sean-k-mooney | im currenly thinking about what to keep and what to factor out into the resoucetracker class or whatever i end up calling it | |
| 12:04:03 | sean-k-mooney | but looking a the current manager ther eare clearly 2 types of functiosn and data | |
| 12:04:16 | gibi | yes | |
| 12:04:30 | gibi | that run method makes it an active component, but also it encapsulates some response data | |
| 12:05:01 | sean-k-mooney | ya so run sleep and generating the responce feel like tehy should all be on one class | |
| 12:05:19 | sean-k-mooney | and the data storage and querying its state shoudl be in anohter | |
| 12:06:16 | sean-k-mooney | gibi: i did not quite understand what you ment by https://review.opendev.org/c/openstack/nova/+/825015/3/nova/healthcheck/manager.py#31 | |
| 12:08:15 | sean-k-mooney | i always split imports into 8 groups , standar lib, external lib, openstack libs, nova and in each of those 4 i split into imports then from x import | |
| 12:09:33 | sean-k-mooney | were you saying that i have split it into more groups then you were expecting or something else? | |
| 12:10:24 | gibi | sean-k-mooney: I meant that in nova I see 3 groups: stdlib, 3pp lib, repo local import | |
| 12:10:35 | gibi | sean-k-mooney: but I have no real problem with more groups | |
| 12:11:16 | gibi | so feel free to ignore that command | |
| 12:11:19 | gibi | *comment | |
| 12:20:36 | sean-k-mooney | gibi: i use the same groups in nova for my other work | |
| 12:21:03 | sean-k-mooney | hacking allows both i really hate when we mix from and import in the same group | |
| 12:21:25 | gibi | OK, I can adapt to it per file :) | |
| 12:21:30 | sean-k-mooney | i used to go fix that but now i more or less deal with the impoort confution | |