| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-21 | |||
| 20:48:13 | sean-k-mooney | dansmith: origially i was calling the osc version "openstack server recreate" but aspiers asked me to change it to "openstack server resurrect" | |
| 20:48:22 | sean-k-mooney | which is what the current patch uses | |
| 20:49:02 | dansmith | sean-k-mooney: right and I'm saying having the command name be something new may not help with the confusion | |
| 20:49:25 | sean-k-mooney | dansmith: that is vaild | |
| 20:49:34 | dansmith | because now it will be "resurrect in osc, which is evacuate in novaclient, which is not host-evacuate or host-evacuate-live" | |
| 20:49:52 | sean-k-mooney | at least for osc we currently dont have the host versions | |
| 20:50:30 | sean-k-mooney | we did talk about that in the fourm session however but bauzas is not a fan of adding them to osc and well i dont care one way or another | |
| 20:50:51 | mriedem | and then i had to yell at both of you, it was great fun | |
| 20:51:14 | sean-k-mooney | :) | |
| 20:52:00 | sean-k-mooney | anyway ill add the docs and a release not and fix all the spelling mistakes tomorrow then let the patch sit for a few days for peopel to think about | |
| 20:57:16 | mriedem | fwiw in osc the sub-command is scoped to the source which might avoid some confusion | |
| 20:57:22 | mriedem | i.e. nova evacuate vs openstack server evacuate | |
| 20:57:39 | mriedem | so that should lessen confusion over what evacuate means per resource | |
| 20:58:00 | mriedem | if we did a whole host evacuate in osc, it would likely be "openstack compute service evacuate" | |
| 20:58:04 | mriedem | since there is a 'compute service' resource | |
| 20:59:08 | sean-k-mooney | is ironic the only case where there is not a 1:1 correspondece between compute services and hypervisors | |
| 20:59:42 | mriedem | yes | |
| 20:59:43 | sean-k-mooney | in the ironic case you would jsut use teh server version | |
| 21:00:00 | sean-k-mooney | ok i was wondering if vmware worked similarly | |
| 21:00:48 | mriedem | it used to back before i think kilo or liberty | |
| 21:00:59 | mriedem | 1 nova-compute service host could manage M vcenters | |
| 21:01:16 | mriedem | much hilarity ensued | |
| 21:01:30 | sean-k-mooney | im sure cdent has fond memories | |
| 21:01:46 | mriedem | this was pre-cdent in nova or vmware | |
| 21:01:59 | mriedem | but coincidentally had dire consequences for evacuate | |
| 21:02:09 | mriedem | i.e. nova deleting all vms in your vcenter | |
| 21:02:15 | mriedem | oops! | |
| 21:02:18 | sean-k-mooney | i can imagine | |
| 21:04:27 | dansmith | mriedem: you mean the scoping actually makes "evacuate" less bad right? | |
| 21:06:02 | sean-k-mooney | the only thing that is unintutive about evacuate (ignoring the host versions) is the fact for ephmereal root volumes you loose the data | |
| 21:07:00 | sean-k-mooney | if evacuated only worked for instacnes with shared storage it actully would be a good name for what it does | |
| 21:08:31 | sean-k-mooney | we brefliy discussed the idea of one command that always preservs your data and a second that always recreates the vm with the same image and flavor but again that might be more confusing | |
| 21:09:48 | sean-k-mooney | anway night all o/ | |
| 21:09:53 | dansmith | that is not the only thing | |
| 21:10:00 | dansmith | but we've been over it a hundred times | |
| 21:10:14 | mriedem | dansmith: yes | |
| 21:10:15 | dansmith | I think keeping evacuate and relying on the scoping to disambiguate is the best plan at the moment | |
| 21:18:59 | sean-k-mooney | ill rephase while i wait for mincraft to load. the only thing where i have gone that must be a bug is the differecne between what we do for instance on shared storage vs non shared storage | |
| 21:42:40 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527 | |
| 22:01:26 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove nova.compute.*API() shims https://review.opendev.org/660527 | |
| 22:01:55 | dansmith | efried: any luck raising melwitt for that ironic power state spec? | |
| 22:03:16 | efried | nope | |
| 22:03:45 | dansmith | aight, I'm about to end my day, so I'll leave it up on my screen to check in the morning | |
| 22:03:52 | efried | cool beans | |
| 22:04:12 | efried | dansmith: FYI I'm +2 all the way on cellsv1 removal, in case you want to hit those... | |
| 22:04:32 | efried | ...for some loose definition of "want", | |
| 22:04:38 | dansmith | efried: probably should wait a few weeks until all the inactive former cellsv1 people have had a chance to look | |
| 22:06:03 | efried | Vek: ^ | |
| 22:11:01 | efried | bauzas: Would you please ack https://review.opendev.org/#/c/651681/ (since you had an earlier -1 on it)? | |
| 22:53:25 | openstackgerrit | John Garbutt proposed openstack/nova master: WIP: Integrating with unified limits https://review.opendev.org/615180 | |
| 22:53:41 | efried | o/ | |
| 22:53:50 | prometheanfire | http://logs.openstack.org/54/660254/2/check/cross-nova-functional/a0617a9/testr_results.html.gz | |
| 22:54:02 | prometheanfire | failures for the new fasteners fasteners===0.15 | |
| #openstack-nova - 2019-05-22 | |||
| 01:09:49 | openstackgerrit | Guo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/659703 | |
| 03:05:28 | openstackgerrit | Merged openstack/nova-specs master: Support server power state update through external event https://review.opendev.org/636132 | |
| 03:31:45 | openstackgerrit | Merged openstack/nova stable/stein: Fix assert methods in unit tests https://review.opendev.org/657708 | |
| 05:18:07 | openstackgerrit | Abhishek Kekane proposed openstack/nova-specs master: Support multiple store of Glance https://review.opendev.org/641210 | |
| 05:21:02 | openstackgerrit | Boxiang Zhu proposed openstack/nova-specs master: Update "add host and hypervisor_hostname flag to create server" spec https://review.opendev.org/660560 | |
| 05:47:19 | boxiang | alex_xu: hi alex, if it's convenient of you, please take a look at this updated spec https://review.opendev.org/#/c/660560/1 | |
| 05:47:26 | boxiang | you can find some comments here https://review.opendev.org/#/c/645520/19/nova/compute/api.py@981 | |
| 05:47:33 | boxiang | thank you :) | |
| 05:48:06 | boxiang | if others here, welcome :) | |
| 07:19:12 | openstackgerrit | Yongli He proposed openstack/nova-specs master: add spec "show-server-numa-topology" https://review.opendev.org/658716 | |
| 07:24:55 | openstackgerrit | zhaixiaojun proposed openstack/nova master: Update description of valid whitelist for non-admin user https://review.opendev.org/642403 | |
| 08:47:02 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/572790 | |
| 08:47:57 | lyarwood | stephenfin: ^ updated the releasenote, can't seem to build it locally but that appears to be an issue with f29 | |
| 09:02:15 | stephenfin | lyarwood: Ta. It reads rather oddly though. Can you take one more swing at it (comments left)? | |
| 09:04:51 | lyarwood | stephenfin: yeah of course | |
| 09:07:16 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/572790 | |
| 09:07:21 | lyarwood | stephenfin: words-- ^ there you go. | |
| 09:09:04 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Block swap volume on volumes with >1 rw attachment https://review.opendev.org/572790 | |
| 09:09:17 | stephenfin | Thanks | |
| 09:09:38 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Keep attach_mode as top-level field in _translate_attachment_ref https://review.opendev.org/574413 | |
| 09:10:44 | lyarwood | haha fs, sorry | |
| 09:50:11 | openstackgerrit | Boxiang Zhu proposed openstack/nova-specs master: Update "add host and hypervisor_hostname flag to create server" spec https://review.opendev.org/660560 | |
| 09:50:48 | stephenfin | bauzas: Wanna help remove nearly 13,000 lines of dead code? :) https://review.opendev.org/#/q/topic:bp/remove-cells-v1+status:open (ignore the ones in merge conflict, obviously) | |
| 10:01:53 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.opendev.org/651307 | |
| 10:01:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove cells v1 options, group https://review.opendev.org/651310 | |
| 10:01:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.opendev.org/651309 | |
| 10:01:54 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.opendev.org/651308 | |
| 10:08:48 | openstackgerrit | Merged openstack/nova stable/stein: Add regression test for bug 1825034 https://review.opendev.org/657420 | |
| 10:08:49 | openstack | bug 1825034 in OpenStack Compute (nova) stein "listing deleted servers from the API fails after running fill_virtual_interface_list online data migration" [High,In progress] https://launchpad.net/bugs/1825034 - Assigned to Matt Riedemann (mriedem) | |
| 10:09:01 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove cells v1 parameter from 'ComputeTaskAPI.resize_instance' https://review.opendev.org/651311 | |
| 11:13:06 | cdent | welcome back gibi | |
| 11:13:14 | gibi | cdent: thanks | |
| 12:01:32 | openstackgerrit | Hamdy Khader proposed openstack/os-vif master: OVS DPDK port representors support https://review.opendev.org/658786 | |
| 12:12:39 | openstackgerrit | Jean-Philippe Evrard proposed openstack/nova master: [Docs] Fix minor typo https://review.opendev.org/660696 | |
| 12:25:25 | sean-k-mooney | stephenfin: bauzas can ye add https://review.opendev.org/#/c/653506/ to your review queue | |
| 12:25:34 | bauzas | mmm ok | |
| 12:31:54 | sean-k-mooney | alex_xu: not sure if you usually review networking/libvirt livemigrtaion stuff but https://review.opendev.org/#/c/653506/ might be in your area too. | |
| 12:46:19 | abhi89 | HI.. i am getting this error in my logs.. | |
| 12:46:20 | abhi89 | The [cinder] section of your nova configuration file must be configured for authentication with the block-storage service endpoint.: InstanceNotFound: Instance 50C28C76-6E6B-4BF0-8291-FBD8C176B9B2 could not be found. | |
| 12:46:43 | abhi89 | it is coming from | |
| 12:46:44 | abhi89 | 2019-05-06 08:58:32.792 123932 ERROR nova.compute.manager [instance: 50c28c76-6e6b-4bf0-8291-fbd8c176b9b2] raise cinder_exception.Unauthorized(401, message=err_msg) | |
| 12:46:44 | abhi89 | File "/usr/lib/python2.7/dist-packages/nova/volume/cinder.py", line 78, in _load_auth_plugin | |
| 12:47:18 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Exec systemd-run without --user flag in Quobyte driver https://review.opendev.org/660705 | |
| 12:48:09 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Added mount fstype based validation of Quobyte mounts https://review.opendev.org/660706 | |
| 12:48:53 | abhi89 | this is very rare.. can anyone help me here to understand do we really need to add 'auth' in [cinder] section of nova.conf..? because without this also deploys work fine | |
| 12:54:36 | efried | abhi89: What release are you on? | |
| 12:54:54 | abhi89 | efried: stein | |