| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-29 | |||
| 19:26:31 | sean-k-mooney | im restacking in offline mode (with libvirt) we are expecting to see the defaulting to ... message if the compute node object is setting the defaults right | |
| 19:28:34 | sean-k-mooney | i can deploy a xen node tommorow if needed to see if i can reporduce | |
| 19:41:30 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert "libvirt: add method to configure migration speed" https://review.openstack.org/590814 | |
| 19:43:53 | cfriesen | jaypipes: re the cold migration with PCI devices. were you talking about the difference between it being theoretically supported and actually doing it? StarlingX integration tests do cold migration with PCI/SRIOV regularly, but I realize that doesn't answer the question for upstream. | |
| 19:45:34 | sean-k-mooney | cfriesen: i think i have done it in the past also i had tought it was ment to be supported. that said not sure it updated teh resouce tracker correctly | |
| 19:45:49 | jaypipes | cfriesen: yes, I'm referring to real-world deployments who do migrations where the instances hold on to their IP addresses, GPUs, and everything else and are migrated to a different rack/region whatever | |
| 19:46:19 | jaypipes | cfriesen: but whatever, I'm running from that conversation screaming. | |
| 19:46:25 | cfriesen | jaypipes: :) | |
| 19:55:35 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for live migrate with anti-affinity group https://review.openstack.org/588935 | |
| 19:56:22 | mriedem | cfriesen: upstream supports cold migration with pci devices | |
| 19:56:31 | mriedem | remember moshe and ludovic got that working | |
| 19:57:03 | mriedem | there was also 3rd party ci from mellanox for it at one time i think | |
| 19:57:06 | mriedem | but that might be dead now | |
| 20:16:11 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | |
| 20:16:12 | mriedem | melwitt: just noticed you had commented on this ^ test should cover the case you noted now | |
| 20:17:08 | melwitt | ok, will look | |
| 20:20:18 | mriedem | crap forgot to update the bug reference in the commit message | |
| 20:21:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete instance_group_member records from API DB during archive https://review.openstack.org/588943 | |
| 20:28:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove old check_attach version check in API https://review.openstack.org/588348 | |
| 20:39:26 | dansmith | melwitt: mriedem: this is going to pass tests in a few minutes: https://review.openstack.org/#/c/597206 | |
| 20:39:49 | dansmith | if you could ack it with a +1 (or tell me what you want changed), I will go about trying to figure out how I'm going to get that merged :) | |
| 20:40:06 | melwitt | will do | |
| 20:43:05 | dansmith | also i was going to verify resource providers before/after and then realized we can't really do that since other projects might create providers, and we have no "service type" field | |
| 20:43:37 | mriedem | one hack way to determine a compute node provider is via the VCPU inventory | |
| 20:44:25 | dansmith | for the moment, yeah, but meh | |
| 20:44:31 | dansmith | I'd rather get this in and work on the other stuff | |
| 20:44:40 | dansmith | because this was a PITA to get working | |
| 20:44:56 | mriedem | dansmith: need to recheck https://review.openstack.org/#/c/597566/ ? | |
| 20:44:57 | dansmith | just because I don't want to run it locally | |
| 20:45:08 | dansmith | mriedem: no, it's about to pass soon too | |
| 20:45:12 | mriedem | ok | |
| 20:45:53 | zigo | mriedem: As I told you, if you wish, I can push your patches into the packages... | |
| 20:45:54 | zigo | Package is building with the patch... | |
| 20:46:19 | mriedem | zigo: sure, but that's not something you'll release is it? with the debug log patch? | |
| 20:46:38 | mriedem | i'm just hoping to debug a recreate with ci logs | |
| 20:47:00 | zigo | mriedem: It just lives in my Stretch backport, until I remove the patch. | |
| 20:47:12 | zigo | I don't have the intention to push that to Debian Sid / Experimental, no. | |
| 20:48:18 | zigo | mriedem: once the package is built by my jenkins (you can see the build process there: https://stretch-queens.infomaniak.ch/job/nova/) then we just need to re-trigger the puppet-openstack CI. | |
| 20:50:18 | zigo | mriedem: Otherwise, I can teach you how to re-produce it on a local Stretch VM. It's very easy . | |
| 20:53:44 | mriedem | that's ok, looks like we have a recreate again in the xen ci https://review.openstack.org/#/c/597613/ | |
| 20:56:29 | mriedem | that doesn't have the logging i need though, so rechecking the xenserver ci job | |
| 20:57:57 | zigo | Silly me, wrong jenkins ... | |
| 20:59:23 | melwitt | dansmith: are you intentionally not checking for DISK_GB in the verify inventory step? | |
| 20:59:49 | dansmith | melwitt: um, duh, of course I'm not | |
| 20:59:50 | dansmith | I mean | |
| 20:59:55 | dansmith | who would verify DISK_GB | |
| 21:00:00 | dansmith | that'd be kinda, like, really stupid right? | |
| 21:00:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add encrypted volume support to feature matrix docs https://review.openstack.org/570255 | |
| 21:00:45 | melwitt | lol | |
| 21:02:19 | dansmith | melwitt: like six of those patchsets were me getting resource classes wrong | |
| 21:02:24 | dansmith | melwitt: just pushed to use a central list | |
| 21:02:45 | melwitt | heh. central list = good | |
| 21:03:20 | dansmith | since it takes about 90 minutes to test each one, I've been trying to make minimal change | |
| 21:03:40 | dansmith | you better hope this one works and I don't have to spend another couple days throwing things at the wall :) | |
| 21:04:08 | melwitt | it's gotta work | |
| 21:18:03 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.openstack.org/469675 | |
| 21:23:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Time how long select_destinations() takes in conductor https://review.openstack.org/517108 | |
| 21:24:42 | zigo | mriedem: Package built, waiting for recheck now. | |
| 21:25:03 | zigo | It probably will end when I'll be sleeping ... | |
| 21:31:29 | mriedem | yeah i'm t-15 minutes from parenting duties | |
| 21:40:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Combine error handling blocks in _do_build_and_run_instance https://review.openstack.org/545960 | |
| 21:50:50 | Sundar | efried: Please ping me | |
| 22:07:10 | openstackgerrit | Merged openstack/nova master: doc: add info how to troubleshoot vmware specific problems https://review.openstack.org/597446 | |
| 23:09:50 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Document differences and similaries between extra specs and hints https://review.openstack.org/581410 | |
| 23:51:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add TODO note for mox removal https://review.openstack.org/576758 | |
| #openstack-nova - 2018-08-30 | |||
| 00:52:29 | openstackgerrit | Merged openstack/nova master: [placement] Make _ensure_aggregate context not independent https://review.openstack.org/597486 | |
| 01:05:06 | openstackgerrit | Merged openstack/nova master: Add explanatory prefix to post_test_perf output https://review.openstack.org/591850 | |
| 01:12:36 | openstackgerrit | Merged openstack/nova master: Add trait query to placement perf check https://review.openstack.org/592624 | |
| 01:12:42 | openstackgerrit | Merged openstack/nova master: Restart scheduler in TestNovaManagePlacementHealAllocations https://review.openstack.org/597571 | |
| 01:12:52 | openstackgerrit | Merged openstack/nova master: reshaper: Look up provider if not in inventories https://review.openstack.org/585033 | |
| 03:28:25 | openstackgerrit | Merged openstack/nova master: Make get_allocations_for_resource_provider raise https://review.openstack.org/584598 | |
| 03:28:35 | openstackgerrit | Merged openstack/nova master: api-ref: fix volume attachment update policy note https://review.openstack.org/596489 | |
| 04:00:51 | moshele | melwitt: hi | |
| 04:02:51 | moshele | melwitt: did I answer you question on https://review.openstack.org/#/c/595592? It an old legacy bug that was revealed because tripleo started to config th rx/tx queues be default | |
| 05:08:53 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix a failure to format config sample https://review.openstack.org/597986 | |
| 05:20:16 | openstackgerrit | Takashi NATSUME proposed openstack/nova stable/rocky: Fix a broken conf file description in networking doc https://review.openstack.org/597987 | |
| 06:21:13 | openstackgerrit | huanhongda proposed openstack/nova master: [WIP]Forbidden non-admin user to list deleted instances https://review.openstack.org/598012 | |
| 07:56:01 | openstackgerrit | Merged openstack/nova stable/rocky: Fix a broken conf file description in networking doc https://review.openstack.org/597987 | |
| 08:18:14 | bauzas | good morning Nova | |
| 08:24:43 | openstackgerrit | huanhongda proposed openstack/nova master: Fix instance delete stuck in deleting task_state https://review.openstack.org/598084 | |
| 08:29:39 | gibi | bauzas: welcome back | |
| 08:29:56 | bauzas | gibi: thanks | |
| 08:42:17 | openstackgerrit | Rong Han proposed openstack/nova master: Reset global variable after unit test is completed. https://review.openstack.org/598088 | |
| 08:55:06 | lyarwood | bauzas: welcome back o/ | |
| 08:55:15 | bauzas | lyarwood: thanks | |
| 08:55:59 | lyarwood | bauzas: https://review.openstack.org/#/q/topic:bug/1787606 - would you mind sticking that on your review queue if you have time today or tomorrow? | |
| 08:56:12 | dpawlik | Hello, which rules from policy.json are used by placement api? I have an another role for admin and its raising me an error on compute host that "ailed to retrieve resource provider tree from placement API for UUID 573c492d-7387-4a3a-b21c-20ce531eb483. Got 403: {"errors": [{"status": 403, "request_id": "req-50ecb39c-8bea-4f52-85f2-7b92db9ae9cf", "detail": "Access was denied to this resource.\n\n admin required ", "title" | |
| 08:56:13 | dpawlik | : "Forbidden"}]}. | |
| 08:56:18 | lyarwood | after rm -rf'ing all of your emails obviously | |
| 08:56:19 | bauzas | lyarwood: today could be difficult but I can try | |
| 08:56:25 | lyarwood | bauzas: yeah no rush | |
| 08:56:38 | bauzas | lyarwood: rm -rf is one option, the other involves reading | |
| 08:56:43 | gibi | sean-k-mooney, melwitt: fyi, we are planning to show some demo on the PTG about the state of the bandwidth work http://lists.openstack.org/pipermail/openstack-dev/2018-August/134015.html | |
| 08:56:46 | lyarwood | bauzas: both are WIP I just wanted to get some input on the bug and potential fix | |
| 08:56:48 | dpawlik | problem is that I changed in policy.json file that admin_api is role:my_role | |
| 08:56:51 | bauzas | lyarwood: I'm not sure which one is the best | |
| 08:56:54 | dpawlik | but its not working on queens | |
| 08:57:02 | lyarwood | bauzas: ^_^ rm -rf every time | |
| 08:57:16 | lyarwood | bauzas: if it's that important people will send another email | |
| 08:57:24 | bauzas | and wait for others yelling at you that you haven't replied them ? That could work | |