| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-02-20 | |||
| 14:19:15 | openstackgerrit | Merged openstack/nova master: Use tempest-full-py3 as base job https://review.opendev.org/708548 | |
| 14:21:37 | openstackgerrit | Merged openstack/nova stable/ocata: Use stable constraint for Tempest pinned stable branches https://review.opendev.org/706872 | |
| 14:44:34 | jroll | stephenfin: <3 thanks for picking that up | |
| 14:51:58 | alex_xu | gmann: gibi lyarwood sean-k-mooney, I guess we need to show the 'delete_on_termination' in GET method also, otherwise I don't know how the user know the current value for delete_on_termination. | |
| 14:52:06 | alex_xu | brinzhang__: ^ | |
| 14:52:40 | sean-k-mooney | am yes we could add that in the same microverion bump | |
| 14:52:55 | alex_xu | for PATCH, I just feel our existing PUT is behavior like PATCH, now we add another PATCH, then we add another mess :) | |
| 14:52:57 | brinzhang__ | alex_xu: agree, if the volume was speficied delete_on_termination, it shuold show | |
| 14:53:38 | alex_xu | I also remember ironic's patch method use json-patch in the body, gmann do you know is there any defintion from the api-wg for the patch's body? | |
| 14:53:42 | sean-k-mooney | alex_xu: well PUT required the entire resoce to be updated. its not sematiclly correct to only set some filed in a PUT | |
| 14:53:53 | sean-k-mooney | partail updates should have been using PATCH | |
| 14:54:15 | alex_xu | sean-k-mooney: oh, yea, right. | |
| 14:54:19 | sean-k-mooney | so the current way we use PUT for swap volume is not correct | |
| 14:54:29 | alex_xu | right | |
| 14:54:30 | sean-k-mooney | but we cant really change that at this point | |
| 14:54:35 | gmann | alex_xu: GET server has that. | |
| 14:55:32 | gmann | https://github.com/openstack/nova/blob/08b98d75cf4b77563e373709a84685f7d0b72ba3/nova/api/openstack/compute/views/servers.py#L716 | |
| 14:55:43 | sean-k-mooney | kevinz: hi do you want to talk about arm testing here or on #openstack-qa | |
| 14:55:43 | gmann | i think that is from microversion 2.3 | |
| 14:56:08 | sean-k-mooney | #openstack-qa might be better to let the volume discussion happen here | |
| 14:56:11 | alex_xu | oh, wait | |
| 14:56:28 | alex_xu | gmann: brinzhang__ we already return that 2.79 | |
| 14:56:34 | kevinz | sean-k-mooney: OK, we can talk at qa channel | |
| 14:57:21 | gmann | alex_xu: yeah - https://github.com/openstack/nova/blob/08b98d75cf4b77563e373709a84685f7d0b72ba3/nova/api/openstack/compute/views/servers.py#L350 | |
| 14:57:46 | brinzhang__ | alex_xu: yeah, it in microversion 2.3 | |
| 14:59:13 | brinzhang__ | gmann: alex_xu said in https://docs.openstack.org/api-ref/compute/?expanded=create-console-detail,show-console-connection-information-detail,list-all-major-versions-detail,show-details-of-specific-api-version-detail,list-servers-detail,create-server-detail,create-multiple-servers-detail,list-servers-detailed-detail,show-server-details-detail,update-server-detail,delete-server-detail,add-associate-floating-ip-addfloati | |
| 14:59:14 | brinzhang__ | etail,remove-disassociate-floating-ip-removefloatingip-action-deprecated-detail,remove-security-group-from-a-server-removesecuritygroup-action-detail,rescue-server-rescue-action-detail,resize-server-resize-action-detail,resume-suspended-server-resume-action-detail,revert-resized-server-revertresize-action-detail,start-server-os-start-action-detail,stop-server-os-stop-action-detail,suspend-server-suspend-action-detail, | |
| 14:59:14 | brinzhang__ | ngip-action-deprecated-detail,add-security-group-to-a-server-addsecuritygroup-action-detail,change-administrative-password-changepassword-action-detail,confirm-resized-server-confirmresize-action-detail,create-server-back-up-createbackup-action-detail,create-image-createimage-action-detail,lock-server-lock-action-detail,pause-server-pause-action-detail,reboot-server-reboot-action-detail,rebuild-server-rebuild-action-d | |
| 14:59:19 | brinzhang__ | unlock-server-unlock-action-detail,unpause-server-unpause-action-detail,unrescue-server-unrescue-action-detail,add-associate-fixed-ip-addfixedip-action-deprecated-detail,remove-disassociate-fixed-ip-removefixedip-action-deprecated-detail,force-delete-server-forcedelete-action-detail,restore-soft-deleted-instance-restore-action-detail,show-console-output-os-getconsoleoutput-action-detail,shelve-server-shelve-action-det | |
| 14:59:24 | brinzhang__ | ail,shelf-offload-remove-server-shelveoffload-action-detail,unshelve-restore-shelved-server-unshelve-action-detail,trigger-crash-dump-in-server-detail,get-rdp-console-os-getrdpconsole-action-deprecated-detail,get-serial-console-os-getserialconsole-action-deprecated-detail,get-spice-console-os-getspiceconsole-action-deprecated-detail,inject-network-information-injectnetworkinfo-action-detail,migrate-server-migrate-acti | |
| 14:59:29 | brinzhang__ | on-detail,live-migrate-server-os-migratelive-action-detail,reset-networking-on-a-server-resetnetwork-action-detail,reset-server-state-os-resetstate-action-detail,evacuate-server-evacuate-action-detail,list-security-groups-by-server-detail,show-server-diagnostics-detail,list-ips-detail,show-ip-details-detail,list-all-metadata-detail,create-or-update-metadata-items-detail,replace-metadata-items-detail,show-metadata-item | |
| 14:59:34 | brinzhang__ | -details-detail,create-or-update-metadata-item-detail,delete-metadata-item-detail,list-actions-for-server-detail,show-server-action-details-detail,list-port-interfaces-detail,create-interface-detail,show-port-interface-details-detail,detach-interface-detail,show-server-password-detail,clear-admin-password-detail,list-volume-attachments-for-an-instance-detail,attach-a-volume-to-an-instance-detail,show-a-detail-of-a-vol | |
| 14:59:39 | brinzhang__ | ume-attachment-detail,update-a-volume-attachment-detail,detach-a-volume-from-an-instance-detail,list-flavors-detail,create-flavor-detail,list-flavors-with-details-detail,show-flavor-details-detail,update-flavor-description-detail,delete-flavor-detail,list-flavor-access-information-for-given-flavor-detail,add-flavor-access-to-tenant-addtenantaccess-action-detail,remove-flavor-access-from-tenant-removetenantaccess-actio | |
| 14:59:41 | alex_xu | :) | |
| 14:59:46 | brinzhang__ | n-detail,list-extra-specs-for-a-flavor-detail,create-extra-specs-for-a-flavor-detail,show-an-extra-spec-for-a-flavor-detail,update-an-extra-spec-for-a-flavor-detail,delete-an-extra-spec-for-a-flavor-detail,list-keypairs-detail,create-or-import-keypair-detail,show-keypair-details-detail,delete-keypair-detail,show-rate-and-absolute-limits-detail,list-agent-builds-detail,create-agent-build-detail,update-agent-build-detai | |
| 14:59:51 | brinzhang__ | l,delete-agent-build-detail,list-aggregates-detail,create-aggregate-detail,show-aggregate-details-detail,update-aggregate-detail,delete-aggregate-detail,add-host-detail,remove-host-detail,create-or-update-aggregate-metadata-detail,request-image-pre-caching-for-aggregate-detail,create-assisted-volume-snapshots-detail,delete-assisted-volume-snapshot-detail,get-availability-zone-information-detail,get-detailed-availabili | |
| 14:59:56 | brinzhang__ | ty-zone-information-detail,list-hypervisors-detail,list-hypervisors-details-detail,show-hypervisor-statistics-detail,show-hypervisor-details-detail,show-hypervisor-uptime-detail,search-hypervisor-detail,list-hypervisor-servers-detail,list-server-usage-audits-detail,list-usage-audits-before-specified-time-detail,list-migrations-detail,id320-detail,show-migration-details-detail,force-migration-complete-action-force-comp | |
| 15:00:01 | brinzhang__ | lete-action-detail,delete-abort-migration-detail,show-a-quota-detail,update-quotas-detail,revert-quotas-to-defaults-detail,list-default-quotas-for-tenant-detail,show-the-detail-of-quota-detail,show-the-quota-for-quota-class-detail,create-or-update-quotas-for-quota-class-detail,list-server-groups-detail,create-server-group-detail,show-server-group-details-detail,delete-server-group-detail,list-tags-detail,replace-tags- | |
| 15:00:06 | brinzhang__ | detail,delete-all-tags-detail,check-tag-existence-detail,add-a-single-tag-detail,delete-a-single-tag-detail,list-compute-services-detail,disable-scheduling-for-a-compute-service-detail,disable-scheduling-for-a-compute-service-and-log-disabled-reason-detail,enable-scheduling-for-a-compute-service-detail,update-forced-down-detail,update-compute-service-detail,delete-compute-service-detail,list-tenant-usage-statistics-fo | |
| 15:00:13 | brinzhang__ | r-all-tenants-detail,show-usage-statistics-for-tenant-detail,run-events-detail,show-server-topology-detail,list-extensions-detail,show-extension-details-detail,list-networks-detail,create-network-detail,add-network-detail,show-network-details-detail,delete-network-detail,associate-host-detail,disassociate-network-detail,disassociate-host-detail,disassociate-project-detail,list-volumes-detail,create-volume-detail,list- | |
| 15:00:18 | brinzhang__ | volumes-with-details-detail,show-volume-details-detail,delete-volume-detail,list-snapshots-detail,create-snapshot-detail,list-snapshots-with-details-detail,show-snapshot-details-detail,delete-snapshot-detail,list-images-detail,list-images-with-details-detail,show-image-details-detail,delete-image-detail,list-image-metadata-detail,create-image-metadata-detail,update-image-metadata-detail,show-image-metadata-item-detail | |
| 15:00:23 | brinzhang__ | ,create-or-update-image-metadata-item-detail,delete-image-metadata-item-detail,list-bare-metal-nodes-detail,show-bare-metal-node-details-detail,list-project-networks-detail,create-project-network-detail,show-project-network-details-detail,delete-project-network-detail,list-floating-ip-pools-detail,list-floating-ip-addresses-detail,create-allocate-floating-ip-address-detail,show-floating-ip-address-details-detail,delet | |
| 15:00:28 | brinzhang__ | e-deallocate-floating-ip-address-detail,list-security-groups-detail,create-security-group-detail,show-security-group-details-detail,update-security-group-detail,delete-security-group-detail,create-security-group-rule-detail,delete-security-group-rule-detail,list-hosts-detail,show-host-details-detail,update-host-status-detail,reboot-host-detail,shut-down-host-detail,start-host-detail,create-root-certificate-detail,show | |
| 15:00:33 | brinzhang__ | -root-certificate-details-detail,list-cloudpipes-detail,create-cloudpipe-detail,update-cloudpipe-detail,ping-instances-detail,ping-an-instance-detail,list-virtual-interfaces-detail,show-fixed-ip-details-detail,reserve-or-release-a-fixed-ip-detail,list-floating-ips-detail,create-floating-ips-detail,bulk-delete-floating-ips-detail,list-floating-ips-by-host-detail,list-dns-domains-detail,create-or-update-dns-domain-detai | |
| 15:00:38 | brinzhang__ | l,delete-dns-domain-detail,list-dns-entries-detail,find-unique-dns-entry-detail,create-or-update-dns-entry-detail,delete-dns-entry-detail,list-cells-detail,create-cell-detail,capacities-detail,list-cells-with-details-detail,info-for-this-cell-detail,show-cell-data-detail,update-a-cell-detail,delete-a-cell-detail,show-cell-capacities-detail,lists-consoles-detail,id562-detail,show-console-details-detail,delete-console-d | |
| 15:00:45 | brinzhang__ | etail,list-default-security-group-rules-detail,show-default-security-group-rule-details-detail,create-default-security-group-rule-detail,delete-default-security-group-rule-detail,get-vnc-console-os-getvncconsole-action-deprecated-detail#list-volume-attachments-for-an-instance | |
| 15:00:49 | brinzhang__ | gmann, alex_xu: sorry | |
| 15:00:51 | brinzhang__ | gmann: alex_xu said is List volume attachments for an instance API | |
| 15:00:53 | brinzhang__ | alex_xu: gmann said in GET server API, we support show delete_on_determination in microversion 2.3 | |
| 15:01:10 | alex_xu | also in 2.79, you can get that from volume-attachment api | |
| 15:01:18 | brinzhang__ | gmann, alex_xu: It seems that there is not the same API we are disscussing | |
| 15:01:29 | alex_xu | so i'm ok now. it sounds like patch can avoid the mess of put method | |
| 15:02:17 | alex_xu | brinzhang__: I can +2 after you update the spec | |
| 15:02:44 | gmann | alex_xu: 'also in 2.79, you can get that from volume-attachment api' you mean to return in list attachment API also ? | |
| 15:03:04 | openstackgerrit | Merged openstack/nova master: zuul: Add Fedora based jobs to the experimental queue https://review.opendev.org/701231 | |
| 15:03:09 | brinzhang__ | alex_xu: I will update seen-k-mooney's words nit, thanks | |
| 15:03:13 | gmann | i think returning in GET server detail is enough and list only return list of attachment without too much details ? | |
| 15:08:42 | openstackgerrit | Merged openstack/nova-specs master: Support volume local cache https://review.opendev.org/689070 | |
| 15:46:23 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Support re-configure deleted_on_termination in server https://review.opendev.org/580336 | |
| 15:47:18 | brinzhang__ | gibi, alex_xu: Update done of destroy-instance-with-datavolume spec, thanks | |
| 15:55:40 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | |
| 16:04:53 | gibi | brinzhang__: ack, will check soon | |
| 16:05:09 | brinzhang__ | gibi: thanks, good night :) | |
| 16:05:19 | gibi | brinzhang__: good night to you too | |
| 16:31:08 | gibi | brinzhang: sorry but as far as I understood the PATCH method was the agreement on the nova meeting | |
| 16:31:40 | sean-k-mooney | ya that was what i tought too | |
| 16:31:47 | sean-k-mooney | rather then an alternitive | |
| 16:32:06 | gibi | but alex_xu was against PATCH | |
| 16:32:12 | gibi | so we have a bit of a deadlock | |
| 16:32:27 | gibi | http://eavesdrop.openstack.org/meetings/nova/2020/nova.2020-02-20-14.00.log.html#l-82 | |
| 16:33:03 | sean-k-mooney | im kind of nutral on this i prefer patch but i can live with either | |
| 16:33:21 | dansmith | I'm not generally a fan of PATCH | |
| 16:33:36 | gibi | honestly I can live with both PUT and PATCH now. | |
| 16:33:45 | gibi | damn, this is hard. | |
| 16:33:50 | gibi | soo many decisions | |
| 16:33:54 | gibi | :) | |
| 16:35:17 | gmann | i am ok with any new API (PATCH or Action) other than changing existing PUT. | |
| 16:35:32 | gmann | action can be one option then. | |
| 16:43:20 | sean-k-mooney | action would have made sense for swap volume | |
| 16:43:35 | sean-k-mooney | im not sure it makes sense here but it would be an option i guess | |
| 16:45:00 | melwitt | I think the ideal would be move swap volume to an action and let PUT be for changing stuff like delete_on_termination, right? but that turns it into boiling the ocean when the author is just trying to add something simple | |
| 16:45:22 | sean-k-mooney | ya | |
| 16:47:05 | melwitt | on the admin-only aspect, I think any "other" params that might end up being allowed in the PUT API need to be able to be controlled by a new separate policy rule right. so that an operator could leave swap volume admin-only but delete_on_termination could be non-admin | |
| 16:47:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Report storage bus traits https://review.opendev.org/666914 | |
| 16:47:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Update '_get_foo_traits' docstrings https://review.opendev.org/708912 | |
| 16:47:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775 | |
| 16:47:28 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models https://review.opendev.org/666915 | |
| 17:06:02 | openstackgerrit | Merged openstack/nova master: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/706868 | |
| 17:25:54 | sean-k-mooney | alex_xu: do you have any documentation on how to configre a host use pmem with openstack. | |
| 17:26:58 | sean-k-mooney | alex_xu: we would like to test it in the next few weeks to plan for how to support it downstream in the osp product but it appears that the defivcie do not have regions avaiable | |
| 17:27:15 | sean-k-mooney | so we are stuck at the create namespace step | |
| 17:28:20 | sean-k-mooney | alex_xu: if you can point use in the right direction for how to create a regaion i think we should be able to create namespaces and configure nova form that point | |
| 17:29:42 | sean-k-mooney | do we have to cofigure kernel memmap options? memmap=12G!4G | |
| 17:30:14 | sean-k-mooney | or is that just for emulated nvdimms | |
| 17:32:02 | sean-k-mooney | i thnk it is jsut for emulated devices and based on https://docs.pmem.io/persistent-memory/getting-started-guide/creating-development-environments/linux-environments it does not support DevDax mode so that is not what we are looking for i guess | |
| 17:34:16 | sean-k-mooney | ah i might have found what i was looking for https://docs.pmem.io/ndctl-user-guide/managing-regions#enabling-regions | |
| 17:46:34 | sean-k-mooney | alex_xu: actully no that is not what i watn but looking at https://software.intel.com/en-us/articles/quick-start-guide-configure-intel-optane-dc-persistent-memory-on-linux i need to use ipmctl to create the regions on teh device then ndctl can be used to manage the namespaces after that point | |
| 17:54:53 | sean-k-mooney | ok i think i understand how to configre this but i might ping you about it at some other point if we get stuck | |
| 18:05:50 | openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add stable device rescue docs https://review.opendev.org/700837 | |
| 18:36:27 | openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | |