Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-29
19:09:17 melwitt ? so how is xen failing? I thought it was taking the alt path
19:09:47 sean-k-mooney melwitt: it is but taking the alt path is apparently not enough
19:09:57 melwitt oh
19:11:40 sean-k-mooney i have matt's debug patch applied also but im not sure that is going to show where the default values got applied
19:12:58 openstackgerrit Matt Riedemann proposed openstack/nova master: Add debug logs for when provider inventory changes https://review.openstack.org/597560
19:13:05 mriedem cdent: like this? ^
19:18:52 cdent mriedem: yeah, nice. that combined with the other stuff ought to help see the flow
19:19:18 cdent s/see/better see/
19:19:37 cdent The difficulty with creating an MTC for this makes me anxious
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

Earlier   Later