| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-02-19 | |||
| 21:14:09 | sean-k-mooney | you have to set it in both | |
| 21:14:17 | spatel | i will give it a try.. | |
| 21:14:43 | sean-k-mooney | it will not work if you are using neutron sriov port e.g. vnic_type=direct | |
| 21:15:10 | sean-k-mooney | it will only be applied to pci devices that are requested vai an alias in the flavor | |
| 21:15:29 | sean-k-mooney | neutron sriov only supports legacy before ussuri | |
| 21:18:17 | spatel | sean-k-mooney: ohhhh! | |
| 21:18:28 | spatel | i am using vnic_type=direct | |
| 21:18:58 | sean-k-mooney | yes the abiltiy to chagne that was only added in ussuri upstream | |
| 21:19:23 | sean-k-mooney | i have backported it downstream but this is not a candiate for backporting upstream | |
| 21:21:13 | sean-k-mooney | spatel: this https://review.opendev.org/#/c/674072/ is the patch you want | |
| 21:23:51 | spatel | any plan that patch get merge in future release? | |
| 21:24:20 | sean-k-mooney | its merged in master so it will be released in ussuri | |
| 21:24:41 | sean-k-mooney | but its adding a new feature so it cant be backported upstream | |
| 21:24:57 | spatel | sean-k-mooney: totally understand.. | |
| 21:25:32 | sean-k-mooney | downstream we are aboult to do a partial backport with out the support for the image property or notification as we cant backport object changes | |
| 21:25:43 | sean-k-mooney | *able | |
| 21:25:56 | sean-k-mooney | that said we avoid it when we can | |
| 21:26:59 | spatel | make sense to avoid thatif there are lots of moving part | |
| 21:27:20 | spatel | http://openstack-in-production.blogspot.com/2015/08/ept-and-ksm-for-high-throughput.html | |
| 21:27:31 | spatel | What do you think about EPT ? | |
| 21:27:45 | sean-k-mooney | you should alway always have it enabeld | |
| 21:28:15 | spatel | CERN suggesting to disable to gain performance | |
| 21:28:16 | sean-k-mooney | if you turn of extended page tables it almost as bad as disbaling kvm and falling back to qemu | |
| 21:28:58 | spatel | hmm! interesting. | |
| 21:29:12 | sean-k-mooney | our perfomance team messurad a huge performance hit to disabeling for the specter/meltdown mitigations | |
| 21:29:49 | spatel | I have that disabled on compute machine but not on VM. | |
| 21:33:25 | sean-k-mooney | spatel: read https://www.redhat.com/pt-br/blog/understanding-l1-terminal-fault-aka-foreshadow-what-you-need-know specifcally the "Beyond bare metal" section alth the full blog is good too | |
| 21:33:43 | spatel | reading... | |
| 21:42:54 | sean-k-mooney | if you turn of EPT you fallback to shadowtables which will result in significat qemu overhead in the 10s or % range based on there testing when fixing l1tf | |
| 21:43:37 | spatel | Yes, you are right. (also based on your link) | |
| 22:40:42 | efried | dansmith: results for the Enum thing are back. Func passed; one unit failed but I'm not sure if it means we need a migration; and grenade failed but I can't tell if it's related. https://review.opendev.org/#/c/708707/ | |
| 23:34:54 | openstackgerrit | Eric Fried proposed openstack/nova master: Use stock mock https://review.opendev.org/708262 | |
| #openstack-nova - 2020-02-20 | |||
| 00:09:58 | openstackgerrit | Eric Fried proposed openstack/nova master: hacking: force explicit import of python's mock https://review.opendev.org/708768 | |
| 00:18:28 | efried | johnthetubaguy: Are you interested in taking over the coding work for vTPM? | |
| 00:18:51 | efried | The design is pretty well described at this point, and the base enablement code is very close to done. | |
| 00:23:16 | efried | stephenfin, gibi, bauzas, alex_xu, johnthetubaguy (and melwitt, dansmith FYI, though I don't expect you to be there so early): In Thursday's Nova meeting, I would like to consider the two spec freeze exceptions that have been requested. | |
| 00:23:17 | efried | Please attend the meeting if you can. | |
| 00:23:17 | efried | Also, please have a look at the specs and see if you think they're close enough to go. | |
| 00:23:17 | efried | They're the first two on this etherpad: https://etherpad.openstack.org/p/nova-ussuri-planning | |
| 00:38:43 | openstackgerrit | Merged openstack/nova master: Follow-up: Add delete_on_termination to volume-attach API https://review.opendev.org/680655 | |
| 01:05:30 | sean-k-mooney | efried: dansmith while i was fixing the cyborg devstack plugin config generation i decied to fix a few other things too. | |
| 01:05:59 | sean-k-mooney | i have one more patch to write to automate the installation of the cyborg python client | |
| 01:06:20 | sean-k-mooney | which ill do tomorrow and then i can add the DNM nova multinode testing job | |
| 01:06:25 | sean-k-mooney | https://review.opendev.org/#/c/708773/1 | |
| 01:06:55 | sean-k-mooney | that is the top of the chain so far | |
| 01:07:31 | sean-k-mooney | anyway after i fixed nested virt in my vms i can now deploy multinode locally and create vms with cybrog flavors | |
| 01:15:36 | sean-k-mooney | dansmith: efried these are teh local.confs i am using http://paste.openstack.org/show/789773/ | |
| 01:18:17 | sean-k-mooney | ill try and get a multinode tempest-full job running in the morning i implement installtion of the client so i can automate creation fo the device profile. although i cloud also do that with curl if i have too | |
| 02:31:25 | openstackgerrit | Merged openstack/nova master: trivial: Remove unused 'cache_utils' APIs https://review.opendev.org/705652 | |
| 02:33:47 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/708783 | |
| 02:33:47 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/708784 | |
| 02:33:48 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-audit https://review.opendev.org/708785 | |
| 02:33:48 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/708786 | |
| 02:33:49 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/708787 | |
| 02:33:49 | openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788 | |
| 07:32:54 | openstackgerrit | Merged openstack/nova master: Reproduce bug 1862633 https://review.opendev.org/706867 | |
| 07:32:54 | openstack | bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,In progress] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 08:56:54 | stephenfin | efried: It would be better if johnthetubaguy can handle vTPM, but if he can't then I can | |
| 09:19:34 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: setup.cfg: Various Python 3 fixes https://review.opendev.org/708824 | |
| 09:40:05 | bauzas | efried: I can't attend this meeting but I'll leave notes in the etherpad | |
| 09:40:10 | bauzas | thanks for the ping either way | |
| 09:46:13 | kashyap | bauzas: "So, given a tempest.log, how do I find which was the last test to run?" | |
| 09:47:44 | bauzas | kashyap: link of the change? | |
| 09:49:17 | kashyap | bauzas: Actually, I forgot it is a local deployment that a QEMY developer was trying | |
| 10:06:03 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: tox: Configure 'ignore_basepython_conflict' https://review.opendev.org/708830 | |
| 10:06:03 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Remove six https://review.opendev.org/708831 | |
| 10:06:04 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Bump to hacking 2.x https://review.opendev.org/708832 | |
| 10:13:57 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/708745 | |
| 11:07:01 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Don't print user_data for 'nova show' https://review.opendev.org/708850 | |
| 11:16:07 | ivve | hello, i have a nova block device attachments stuck (i have been trying to get it removed with nova and openstack commands but with no avail) i can see it still in the nova.block_device_mapping table, any recommendations on setting it properly to deleted? | |
| 11:16:35 | ivve | other than removing the entire row | |
| 11:53:04 | openstackgerrit | Merged openstack/nova stable/queens: Mask the token used to allow access to consoles https://review.opendev.org/707845 | |
| 12:50:56 | gibi | efried: I will attend the nova meeting today to talk about FFE but I hope we can make it quick as I have a parallel meeting at the same time. | |
| 12:56:45 | sean-k-mooney | gibi: is that for the local cache or delete on terminate spec or both | |
| 12:57:13 | sean-k-mooney | i reviewd the delete on terminate spec and other then some minor wording issue i think its good to go | |
| 12:57:34 | sean-k-mooney | i have not looked at the local caching one in a few days but ill try to take a look before the meeting | |
| 12:58:06 | gibi | sean-k-mooney: efried collecs opinions here https://etherpad.openstack.org/p/nova-ussuri-planning | |
| 12:58:39 | gibi | sean-k-mooney: for the local cache I think we can support that but we might not need to commit to it yet as there are os-brick and cinder dependencies to be implemnted first | |
| 12:58:42 | sean-k-mooney | ah righit i just comemnted on the spec ill add mine | |
| 12:58:57 | sean-k-mooney | gibi: ya | |
| 12:59:15 | sean-k-mooney | gibi: i think cinder were planning to approve there FFE today before the nova meeting | |
| 12:59:18 | gibi | sean-k-mooney: for the delte on terminate I saw an API problem raised from gmann that wasnot resolved | |
| 12:59:55 | sean-k-mooney | ah i did not know he had raised an issue | |
| 13:01:39 | gibi | sean-k-mooney: I'm trying to find the description of the problem | |
| 13:01:43 | sean-k-mooney | gibi: ya so i realy dont know hwy we dont just support setting delete on terminate via a put to /servers/{server_id}/os-volume_attachments/{volume_id} | |
| 13:02:27 | gibi | sean-k-mooney: https://review.opendev.org/#/c/580336/28/specs/ussuri/approved/destroy-instance-with-datavolume.rst@43 | |
| 13:02:39 | sean-k-mooney | ^ was the open question i had. i assume PUT to that endpoing is how we trigger swap volumn? | |
| 13:02:49 | gibi | sean-k-mooney: because that PUT is for swap volume not for updating existing volume | |
| 13:03:22 | sean-k-mooney | ya that is what i was assuming | |
| 13:03:58 | sean-k-mooney | so we are just modifying the put then so that it only swaps the value if you alter the volume id | |
| 13:04:00 | gibi | so we made a suboptimal decision in the past to use PUT as swap and now we have no good place to implement a simple update as PUT is already used | |
| 13:04:06 | sean-k-mooney | which is why it is no optional in the spefc | |
| 13:04:24 | sean-k-mooney | gibi: well we could use patch | |
| 13:04:43 | gibi | sean-k-mooney: I could accept to have a conditional swap as you described above | |
| 13:04:49 | gibi | sean-k-mooney: I'm not sure gmann likes that | |
| 13:05:04 | sean-k-mooney | right well the spec currently had the conditional swap | |
| 13:05:20 | sean-k-mooney | was useing PATCH discussed? | |
| 13:06:20 | sean-k-mooney | PATCH is intended for a partial update where you jsut sent the data to be updated so PATCH would just contain delete_on_terminate=True|False in the body and use the volumn id for the url | |
| 13:06:38 | sean-k-mooney | gmann: ^ would you be ok with that? | |
| 13:06:55 | sean-k-mooney | gmann: it would allow put to be kept just for swap volumn | |