Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-10-06
02:15:01 openstackgerrit Merged openstack/nova stable/ussuri: docs: fix aggregate weight multiplier property names https://review.opendev.org/753392
02:15:11 openstackgerrit Merged openstack/nova stable/ussuri: Test for disabling greendns https://review.opendev.org/754220
02:16:44 openstackgerrit Merged openstack/nova stable/stein: post live migration: don't call Neutron needlessly https://review.opendev.org/750674
02:28:18 openstackgerrit Merged openstack/nova stable/train: Sanity check instance mapping during scheduling https://review.opendev.org/752279
02:28:26 openstackgerrit Merged openstack/nova stable/train: Set different VirtualDevice.key https://review.opendev.org/752843
02:28:38 openstackgerrit Merged openstack/nova stable/ussuri: tests: Add regression test for bug 1894966 https://review.opendev.org/752371
02:28:38 openstack bug 1894966 in OpenStack Compute (nova) ussuri "Create servergroup failed with unexpected error" [Undecided,In progress] https://launchpad.net/bugs/1894966 - Assigned to Stephen Finucane (stephenfinucane)
02:28:46 openstackgerrit Merged openstack/nova stable/ussuri: api: Set min, maxItems for server_group.policies field https://review.opendev.org/752702
02:28:55 openstackgerrit Merged openstack/nova master: Update master for stable/victoria https://review.opendev.org/754321
07:44:25 bauzas good morning Nova
08:43:34 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Increase incremental and max sleep time during device detach https://review.opendev.org/755799
08:43:35 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM: Test Ib9ed7069cef5b73033351f7a78a3fb566753970d on Focal https://review.opendev.org/756097
08:54:12 openstackgerrit Lee Yarwood proposed openstack/nova master: fup: Stop returning a retry function from detach_device_with_retry https://review.opendev.org/756084
09:05:24 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: 'video.vram' property must be an integer https://review.opendev.org/753011
09:22:31 bauzas gibi: hola, do we have a documentation knowing which instance actions are supporting QoS minimum bandwidth ?
09:22:51 bauzas gibi: can't see it in either https://docs.openstack.org/nova/latest/admin/ports-with-resource-requests.html or https://docs.openstack.org/neutron/latest/admin/config-qos-min-bw.html
09:28:12 openstackgerrit Takashi Natsume proposed openstack/nova master: Update contributor guide for Wallaby https://review.opendev.org/754427
09:28:55 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224
09:29:08 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757
09:29:19 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777
09:29:45 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094
09:30:00 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109
09:30:22 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117
09:51:40 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Fix issue in exception type https://review.opendev.org/756069
09:51:41 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add tests for mixed CPU policy https://review.opendev.org/755852
09:51:41 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add test for #1898272 https://review.opendev.org/756100
09:51:42 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Check inventory of shared CPUs for 'mixed' policy https://review.opendev.org/756101
09:51:42 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'InstanceNUMACell.min_threads' https://review.opendev.org/756070
09:52:56 gibi bauzas: there is a 3rd place https://docs.openstack.org/api-guide/compute/port_with_resource_request.html
09:53:19 lyarwood tok~.
09:53:25 bauzas gibi: ah thanks, I remembered we documented it somewhere but I forgot about it
09:53:29 lyarwood argh weechat--
09:54:12 bauzas lyarwood: what kind of tiktok video were you about to upload ?
09:54:15 lyarwood tosky: https://review.opendev.org/#/c/755270/10/roles/sync-ceph-conf-and-keys/tasks/main.yaml - could use your advice on this change btw, my ansible foo is failing me here. I'm trying to copy files from a deployed controller to a subnode.
09:54:56 lyarwood bauzas: ^ was trying to ask that question and weechat/screen/ssh borked
09:55:58 bauzas ;-)
09:58:55 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Reproposes Offline Reshape tool spec https://review.opendev.org/742908
10:09:03 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Re-proposes Routed Networks https://review.opendev.org/756242
10:09:51 openstackgerrit Sylvain Bauza proposed openstack/nova-specs master: Offline Reshape tool spec https://review.opendev.org/742908
10:22:16 sean-k-mooney lyarwood: there is an example of this for the ssh keys
10:23:14 sean-k-mooney the copy-build-sshkey role
10:23:37 lyarwood sean-k-mooney: yeah I tried using that approach but wanted to only copy the generated keyrings, iirc that role copies a hardcoded list
10:25:25 sean-k-mooney its defiended here
10:26:05 sean-k-mooney the ssh key it copies is defiend by this var https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/copy-build-sshkey/vars/main.yaml#L1
10:26:12 sean-k-mooney but you could write a similar role
10:26:17 sean-k-mooney for the ceph keys
10:26:41 lyarwood sean-k-mooney: did you not click through to the review? :)
10:26:45 sean-k-mooney this is all you need to do https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/copy-build-sshkey/tasks/main.yaml#L16-L34
10:26:53 sean-k-mooney nope
10:27:23 sean-k-mooney i see
10:27:37 sean-k-mooney shoudl this not be in the ceph devstack plugin repo instead
10:27:39 lyarwood sean-k-mooney: I'm just failing at sharing the list of generated keyrings between the controller and compute, it's a basic ansible thing I'm failing on
10:28:09 lyarwood sean-k-mooney: zuul wasn't happy when I started with it in the plugin
10:28:19 lyarwood sean-k-mooney: so I'm leaving it here until it's working
10:28:31 lyarwood gah baby woke up, brb
10:29:57 sean-k-mooney lyarwood: ok well you could alos look at https://github.com/openstack/devstack/tree/master/roles/sync-devstack-data
10:30:17 sean-k-mooney that copies the tls certs for multinode setups beteween hosts
10:32:16 sean-k-mooney its using the synchronize role instead of fetch and copy
10:47:06 lyarwood sean-k-mooney: yeah also looked at that in previous PS's and got confused so I switched back.
10:47:25 lyarwood sean-k-mooney: my main issue is with how I should store and share the list of keyrings between the hosts
10:47:58 lyarwood sean-k-mooney: hmm do I set it as a fact against the controller and then look that up from the compute?
10:48:06 sean-k-mooney lyarwood: you proably just want to sync all of them to the zuul executor and then all of them to the subnodes
10:48:38 lyarwood sean-k-mooney: right but I don't think any of the copy/sync modules support wildcards do they?
10:48:39 sean-k-mooney so just copy all of them into a singel folder and copy all the files in the folder
10:49:37 sean-k-mooney so you basically want to do https://github.com/openstack/devstack/blob/master/roles/sync-devstack-data/tasks/main.yaml#L21-L30 but replace the with_items
10:49:38 lyarwood k I'll give that a go once my wife gets home and I can throw the baby at her
10:49:56 lyarwood sean-k-mooney: right but the hardcoded list was something I wanted to avoid
10:50:04 sean-k-mooney ya you can
10:50:17 sean-k-mooney you can register the files as a fact on the contoler
10:50:27 sean-k-mooney then use it on the contoler to sync it to the exectutor
10:50:32 lyarwood yup
10:50:44 lyarwood the part I was missing was that it wasn't a local fact on the compute
10:50:46 sean-k-mooney then do the same thing on the executor to push them
10:50:58 lyarwood ansible is weird sometimesd
10:51:15 sean-k-mooney well its sort as a host var
10:51:27 sean-k-mooney most facts are specific to the host they are created on
10:51:57 sean-k-mooney but the point is to pull the data from the contoler to the executor then push it form the executor to the compute nodes
10:52:12 sean-k-mooney you never need to run the sync on the computes
10:53:02 sean-k-mooney if you know the contolers name you can also just use that to get its fact
11:10:39 openstackgerrit Rajat Dhasmana proposed openstack/nova master: WIP: Add support of blockCommit when VM is down https://review.opendev.org/756261
11:18:07 tosky lyarwood: I think sean-k-mooney already gave more information than I could have provided
11:24:13 openstackgerrit Elod Illes proposed openstack/nova stable/stein: Set different VirtualDevice.key https://review.opendev.org/749415
11:33:52 lyarwood tosky: ack thanks
11:41:37 andrewbonney Hi. We're hitting an issue with live migration after moving to Ussuri which I think may be down to nova not handling some output from libvirt. I've put some debug here if anyone could take a look: http://paste.openstack.org/show/798740/. I'd be happy to post that to launchpad if preferred
11:42:36 lyarwood bauzas / melwitt ; https://review.opendev.org/#/q/branch:%255Estable/.*+status:open+owner:%22Lee+Yarwood+%253Clyarwood%2540redhat.com%253E%22+label:Code-Review%253E%253D2 - btw my stable backlog is starting to build up if either of you have time this week
11:48:02 gibi andrewbonney: seems like a valid bug. Could you please file a bug report in launchpad for it https://bugs.launchpad.net/nova/+filebug
11:48:21 andrewbonney Will do, thanks
12:04:23 bauzas lyarwood: ack, will look
13:51:09 lyarwood bauzas: many thanks btw
13:52:35 bauzas lyarwood: np, took a bit to verify all the changes, but thanks for it
13:56:32 lyarwood fun we dump the devstack run from a controller into the job logs twice for multinode jobs at the moment
13:56:36 lyarwood 29871 2020-10-06 12:07:45.753810 | TASK [run-devstack : Run devstack]
13:56:39 lyarwood 29872 2020-10-06 11:42:53.973082 | controller | + unset GREP_OPTIONS
13:57:02 lyarwood ^ that's the compute run with logs from the previously already logged controller run
13:57:07 lyarwood /o\
14:33:35 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Support extending locally attached rbd volumes https://review.opendev.org/752069
16:05:30 openstackgerrit Ghanshyam Mann proposed openstack/nova master: DNM: testing tempest copy-image test https://review.opendev.org/742547
16:17:57 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP: nova-next: Start testing the 'q35' machine type https://review.opendev.org/708701
18:14:56 sean-k-mooney melwitt: can you take a look at https://review.opendev.org/#/c/751642/6 or lyarwood if you are about its relitivly minor but i need it for the follow up patch
18:20:33 dansmith sean-k-mooney: your comment in driver.py is exactly why N->N+m is not just doable when rpc versions match :)

Earlier   Later