Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-05-26
16:23:53 lyarwood that pre-date v2 so we started with luks and then added luksv2 later
16:24:00 lyarwood yeah
16:24:02 sean-k-mooney if the traits are already there then ok
16:24:24 lyarwood cool cool
16:25:13 sean-k-mooney i feel like im +w too many thing so i just +2'd https://review.opendev.org/c/openstack/nova-specs/+/783806
16:25:41 stephenfin +W ALL the things!
16:25:58 stephenfin I did it for you ;)
16:26:21 sean-k-mooney * sean-k-mooney is trying to be conservitive with merge writes
16:26:45 sean-k-mooney * sean-k-mooney is also reading a lot of well trought out specs
16:26:52 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Enable 'vmcoreinfo' feature by default https://review.opendev.org/c/openstack/nova/+/752912
16:26:53 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: libvirt: Make vmcoreinfo checks guest arch specific https://review.opendev.org/c/openstack/nova/+/790946
16:26:53 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Make vmcoreinfo dependent on feature flag https://review.opendev.org/c/openstack/nova/+/779342
16:26:53 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove 'create_fake_libvirt_mock' https://review.opendev.org/c/openstack/nova/+/790945
16:27:15 stephenfin lyarwood: Could I get your +2 on the first of those again? ^
16:27:19 stephenfin merge conflict :(
16:27:37 lyarwood yup no issues
16:33:44 artom sean-k-mooney, so about https://bugzilla.redhat.com/show_bug.cgi?id=1963164 - you said libvirt being restarted will remove and re-add the TAP interface, right?
16:33:45 openstack bugzilla.redhat.com bug 1963164 in openstack-nova "[update] 16.1 to 16.2 update fail on ping loss." [Urgent,New] - Assigned to nova-maint
16:34:02 sean-k-mooney artom: i belive it will trigger https://github.com/libvirt/libvirt/blob/c8238579fb0b1c3affbd77749ae2b2c4dfafb2d6/src/util/virnetdevopenvswitch.c#L158-L159
16:34:17 sean-k-mooney we shoudl get them to confirm if that is the case
16:34:27 artom sean-k-mooney, this is OVN though
16:34:33 sean-k-mooney yes
16:34:43 sean-k-mooney form a libvirt point of view it cant tell teh difference
16:34:54 artom OK, that code looks like OVS, but that's just my ignorance talking
16:35:00 sean-k-mooney ovn is a replamcent for the neutorn l2 agent not a replacment for ovs
16:35:51 artom So we need to get #virt eyes on the BZ then...
16:36:20 sean-k-mooney yes to confirm if restrating will cause the port add to be reporces after a contaienr upgrade
16:36:32 artom (We're in the wrong channel, aren't we?)
16:36:45 sean-k-mooney on 16/traing plug_vif is a noop for ovn
16:36:52 artom This was supposed to be downstream, but with the, err, kerfuffle, I have more networks open than before ;)
16:36:55 sean-k-mooney both work
16:37:27 sean-k-mooney there is noting private related to that
16:37:50 artom I know, just scope-wise
16:38:24 sean-k-mooney scope wise i am 99.99% sure this is not related to nova
16:38:28 sean-k-mooney or os-vif
16:39:03 sean-k-mooney so yes we should follow up with the libvirt folks downstream and get them to confrim
16:44:08 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Remove 'hosts_up' https://review.opendev.org/c/openstack/nova/+/773641
16:44:08 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Remove 'USES_ALLOCATION_CANDIDATES' https://review.opendev.org/c/openstack/nova/+/773640
16:44:09 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Merge 'FilterScheduler' into base class https://review.opendev.org/c/openstack/nova/+/773643
16:44:09 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Remove FakeScheduler (for realz) https://review.opendev.org/c/openstack/nova/+/773642
16:44:10 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Drop references to non-filter scheduler drivers https://review.opendev.org/c/openstack/nova/+/773645
16:44:10 openstackgerrit Stephen Finucane proposed openstack/nova master: scheduler: Merge driver into manager https://review.opendev.org/c/openstack/nova/+/773644
16:44:11 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Remove deprecated aliases https://review.opendev.org/c/openstack/nova/+/773647
16:44:11 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Merge 'test_utils', 'test_scheduler_utils' https://review.opendev.org/c/openstack/nova/+/773646
16:45:04 stephenfin lyarwood: might ask for you to look at https://review.opendev.org/c/openstack/nova/+/773640/ again too, if you could. Again, trivial merge conflict
16:45:17 stephenfin I'll bug bauzas for reviews on that tomorrow and try close it out
16:45:27 stephenfin Or Monday, rather
16:45:33 stephenfin * stephenfin is out for the rest of the week
16:46:26 lyarwood Yup no issues I can hit that again later this evening or tomorrow
16:47:22 stephenfin Perfect, thanks. Let me know if I can return the favour
17:04:57 openstackgerrit Merged openstack/nova-specs master: Repropose flavour and image defined ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/783806
18:36:46 openstackgerrit Lee Yarwood proposed openstack/nova master: hardware: Use image_meta.id within get_mem_encryption_constraint https://review.opendev.org/c/openstack/nova/+/790660
18:36:47 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Skip swap_volume tests as part of nova-next https://review.opendev.org/c/openstack/nova/+/793219
18:36:49 lyarwood ^ giving up trying to fix swap_volume for now as the change above is more important, core reviews would be appreciated.
18:40:17 sean-k-mooney https://review.opendev.org/c/openstack/os-vif/+/792840 has now passed ci
18:41:10 sean-k-mooney stephenfin: ^ are you ok with me continutin gto back port your change
18:49:55 melwitt sean-k-mooney[m]: did you want stephenfin to ack that before +W?
18:50:55 sean-k-mooney[m] melwitt: am i think we can proceed but stephenfin was suggesting that if it does not work we can just delete the job
18:51:10 sean-k-mooney[m] it works so if it comes to it we can still delete the job next time it breaks
18:51:18 melwitt sean-k-mooney[m]: he just wants to delete everything
18:51:23 melwitt jk stephenfin :)
18:51:47 sean-k-mooney[m] im going to backport that to train later today if i can
18:52:00 sean-k-mooney[m] well or tommorow if it gets too late
19:17:11 sean-k-mooney ok it will be tomorow thank melwitt for looking at that
19:18:10 melwitt yw
20:24:54 openstackgerrit melanie witt proposed openstack/nova stable/victoria: Initialize global data separately and run_once in WSGI app init https://review.opendev.org/c/openstack/nova/+/785059
20:41:32 openstackgerrit Merged openstack/os-vif stable/victoria: Resolve dependency issues https://review.opendev.org/c/openstack/os-vif/+/792840
#openstack-nova - 2021-05-27
01:15:27 openstackgerrit norman shen proposed openstack/nova-specs master: Speed up server details https://review.opendev.org/c/openstack/nova-specs/+/791620
01:30:52 openstackgerrit Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/c/openstack/nova-specs/+/787178
02:24:20 openstackgerrit Kevin Zhao proposed openstack/nova master: Support Cpu Compararion on Aarch64 Platform https://review.opendev.org/c/openstack/nova/+/763928
07:53:10 lyarwood gibi: https://review.opendev.org/c/openstack/nova/+/790660 - I just W+'d this again after the rebase btw
07:53:25 lyarwood * lyarwood is getting bored of gate failures this week
07:53:44 lyarwood gibi: https://review.opendev.org/c/openstack/nova/+/793219/ can you take a look at this so ^ can land?
08:10:44 bauzas lyarwood: I can take a look
08:10:50 lyarwood thanks
08:23:11 gibi lyarwood: looking too
08:31:21 openstackgerrit Merged openstack/nova-specs master: QoS minimum guaranteed packet rate https://review.opendev.org/c/openstack/nova-specs/+/785014
08:50:35 bauzas sean-k-mooney: sorry, I didn't wanted to hold the pps spec from gibi when I said 'I'll look at the spec'
08:50:49 bauzas sean-k-mooney: it was just for knowing what was your concern
08:51:05 bauzas anyway, merged now :)
08:57:09 gibi bauzas,sean-k-mooney: thanks
08:57:27 gibi I have feedback from Rodolfo on the neutron side so I will continue with that
09:54:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/wallaby: image_meta: Provide image_ref as the id when fetching from instance https://review.opendev.org/c/openstack/nova/+/793354
10:02:45 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: libvirt: make cross cell resize spawn from snapshot image https://review.opendev.org/c/openstack/nova/+/793356
10:02:46 openstackgerrit Lee Yarwood proposed openstack/nova stable/victoria: image_meta: Provide image_ref as the id when fetching from instance https://review.opendev.org/c/openstack/nova/+/793357
10:26:10 slaweq lyarwood: hi, can You maybe check https://review.opendev.org/c/openstack/nova/+/787252 ?
10:26:12 slaweq thx in advance
10:49:09 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: Add regression test for bug #1928063 https://review.opendev.org/c/openstack/nova/+/793372
10:49:10 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: libvirt: make cross cell resize spawn from snapshot image https://review.opendev.org/c/openstack/nova/+/793373
10:49:11 openstackgerrit Lee Yarwood proposed openstack/nova stable/ussuri: image_meta: Provide image_ref as the id when fetching from instance https://review.opendev.org/c/openstack/nova/+/793374
10:49:15 openstack bug 1928063 in OpenStack Compute (nova) "SEV enabled instance unable to hard reboot" [Medium,In progress] https://launchpad.net/bugs/1928063 - Assigned to Lee Yarwood (lyarwood)
13:51:24 bauzas gibi: gentle reminder that all the Red Hat folks should be off tomorrow
13:52:07 bauzas and like every year, I have this PTO just for my lawn mower...
13:54:30 gibi bauzas: thanks for the reminder
13:54:36 gibi for all RH folks, have a nice day off!
14:02:12 sean-k-mooney :) actully im taking monday off too
14:02:25 sean-k-mooney but thank you none the less
15:20:53 lyarwood * lyarwood is off on both Friday and Monday FWIW
15:34:59 melwitt same for me
15:36:28 melwitt lyarwood: sorry if this has been mentioned before but are you familiar with this failure setting up ceph "Error EPERM: configuring pool size as 1 is disabled by default." this is on stable/victoria https://zuul.opendev.org/t/openstack/build/7de187da76af48aab23337c4f9d16f9c/log/job-output.txt#5403

Earlier   Later