| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-14 | |||
| 15:40:38 | mriedem | yeah, plus that matches what the ironic driver already does today | |
| 15:40:42 | dansmith | yep | |
| 15:41:19 | sean-k-mooney | ya that does seem alot cleaner | |
| 15:41:42 | mriedem | i'm not sure if that happens immediately or in a periodic? when the admin disables the compute service in the api, does the api set reserved=total for all inventory on the compute node resource provider? | |
| 15:42:09 | mriedem | seems it should be immediately mirrored | |
| 15:42:11 | dansmith | disable is an rpc call right? | |
| 15:42:16 | mriedem | no | |
| 15:42:21 | dansmith | because xen used to do stuff when you disabled it I think | |
| 15:42:47 | dansmith | maybe I'm mixing in the maintenance mode call | |
| 15:42:49 | mriedem | it's just a db change | |
| 15:42:53 | mriedem | yeah i think yo uare | |
| 15:43:27 | dansmith | okay, so I was going to say let the compute service do the disable, | |
| 15:43:33 | dansmith | but same difference just having api do it | |
| 15:43:44 | sean-k-mooney | mriedem: well renebaling it would need the virt driver to resore the old reserved value right | |
| 15:43:45 | dansmith | it's more better to let compute do it I think, just because it's the one that owns the resources | |
| 15:43:47 | dansmith | and the RP | |
| 15:43:51 | mriedem | set_host_enabled was the thing you're thinking of | |
| 15:43:54 | mriedem | in the os-hosts api | |
| 15:44:07 | jaypipes | sean-k-mooney, dansmith, mriedem: I'm still reviewing Virgil's Aeneid^W^Wthe Cyborg-nova interaction spec. | |
| 15:44:09 | dansmith | and that doesn't get called when disable? | |
| 15:44:16 | sean-k-mooney | mriedem: what inventor was reserver=total set on by the way | |
| 15:44:19 | mriedem | dansmith: no | |
| 15:44:35 | mriedem | PUT /os-services/{compute_node_id} is just a db change on the services record | |
| 15:44:43 | sean-k-mooney | jaypipes: im reviewing that currently too | |
| 15:44:52 | sean-k-mooney | jaypipes: this one right https://review.openstack.org/#/c/603955/10/specs/stein/approved/nova-cyborg-interaction.rst,unified | |
| 15:44:59 | dansmith | mriedem: okay, well, it's rather unfortunate to have the api muck with the inventory of the compute node, but mechanically it would work | |
| 15:45:13 | jaypipes | sean-k-mooney: yes. that one. | |
| 15:45:19 | dansmith | mriedem: un-disabling is a good thought though, since you'd really not want to do that from the api I think | |
| 15:45:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | |
| 15:45:35 | sean-k-mooney | jaypipes: i jsut pushed what i had it take a while to get through that end to end | |
| 15:45:52 | mriedem | dansmith: you mean fixing the reserved value once the service is re-enabled? | |
| 15:45:58 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | |
| 15:46:16 | dansmith | mriedem: yeah | |
| 15:46:19 | mriedem | yeah i don't think we can do that from api, compute knows that info | |
| 15:46:19 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | |
| 15:46:30 | mriedem | some of it is stored in the compute node record, but maybe not all inventory records | |
| 15:46:33 | mriedem | only ram/vcpu/disk | |
| 15:46:33 | dansmith | mriedem: right.. also there's a potential race I guess, | |
| 15:46:38 | mriedem | yeah | |
| 15:46:51 | mriedem | let the compute b/c the source of truth for inventory (unless the API reserves it all...?) | |
| 15:46:58 | dansmith | yeah | |
| 15:46:58 | dansmith | mriedem: which is api racing with compute to update the inventory.. they could both use generation to catch the race, but probably better to just let one own it | |
| 15:47:16 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add description of custom resource classes https://review.openstack.org/616721 | |
| 15:47:39 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | |
| 15:48:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions for vol-backed snapshots https://review.openstack.org/615084 | |
| 15:48:25 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use links to placement docs in nova docs https://review.openstack.org/614056 | |
| 15:48:39 | sean-k-mooney | jaypipes: are you at the summit by the way. you ususally only attend the ptgs as of late | |
| 15:48:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove Placement API reference https://review.openstack.org/614437 | |
| 15:48:52 | jaypipes | sean-k-mooney: no. I'm not in Berlin | |
| 15:49:08 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629 | |
| 15:49:26 | sean-k-mooney | jaypipes: ah ok same. its weird not being there but also i got to review that spec again | |
| 15:49:28 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use oslo_db.sqlalchemy.test_fixtures https://review.openstack.org/609352 | |
| 15:50:05 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix best_match() deprecation warning https://review.openstack.org/611204 | |
| 15:50:23 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix server query examples https://review.openstack.org/616834 | |
| 15:51:08 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 15:56:15 | jaypipes | sean-k-mooney: hell of a reward... | |
| 16:04:26 | mriedem | ha | |
| 16:50:58 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 16:56:34 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 17:35:36 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 19:45:08 | spatel | sean-k-mooney: ^^ | |
| 19:46:08 | spatel | if i have 8 vCPU on instance then its ok to set ethtool -L eth0 combined 8 ? | |
| 19:46:35 | spatel | or i should set 4 (50% of number of cpu) | |
| 20:32:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send RP uuid in the port binding https://review.openstack.org/569459 | |
| 20:32:45 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Pass allocations and traits to neturonv2 api https://review.openstack.org/616240 | |
| 20:32:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| #openstack-nova - 2018-11-15 | |||
| 00:01:18 | sean-k-mooney | spatel you should set it to 8 assuming you have eneabled multi queue | |
| 02:21:37 | lucidguy | lord help me | |
| 02:22:16 | lucidguy | New installment of OpenStack Queens, everything looks good yet my instances just sit there scheduling. I can't figure it out, logs tell me nothing. | |
| 02:22:19 | lucidguy | help | |
| 02:22:21 | lucidguy | :) | |
| 07:09:27 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/614757 | |
| 10:06:05 | mriedem | bauzas: gibi: do we have a functional test with vGPU resources that verifies reshaping of flat vGPU resources to nested vGPU resources and successful scheduling to the same compute host after reshaping - maybe not necessarily vgpu, but some kind of resource reshape + scheduling | |
| 10:06:15 | mriedem | i thought that was being worked on during the ptg by tetsuro | |
| 10:15:02 | gibi | mriedem: let me dig a bit | |
| 10:15:57 | gibi | mriedem: at least I did not implemented such scenario | |
| 10:18:49 | melwitt | yeah, looks like it would need to added to this (reshaper for vGPU) patch or stacked on top https://review.openstack.org/599208 | |
| 10:20:25 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 10:21:45 | gibi | mriedem, melwitt: there is nova.tests.functional.test_servers.ProviderTreeTests.test_reshape it almost what you want. it boots, then reshapes but it does not boot again after the reshape | |
| 10:22:10 | mriedem | ok i thought we had a functional test to go from flat to nested and then schedule another workload on that provider | |
| 10:22:49 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 10:23:55 | cdent | gibi: that ^ is the latest. I've needed to do quite a bit to make unit tests not think about placement | |
| 10:25:01 | melwitt | gibi: oh, so that same test will exercise the virt integration code? that's where I'm not clear on | |
| 10:26:14 | gibi | cdent: ack | |
| 10:27:50 | gibi | melwitt: to have rehape in the test we need a virt layer returning a reshaped tree. I don't think the actal vgpu virt integration needs to be covered in the same test | |
| 10:28:56 | melwitt | what I mean is, does test_reshape verify that https://review.openstack.org/599208 works correctly I wonder | |
| 10:32:19 | gibi | melwitt: it does not cover the VGPU resource specifically therefore it does not cover the change in https://review.openstack.org/#/c/599208/8/nova/virt/libvirt/driver.py | |
| 10:33:49 | melwitt | thanks | |
| 10:49:46 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 11:22:08 | openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Use external placement in functional tests https://review.openstack.org/617941 | |
| 11:23:00 | sean-k-mooney | stephenfin: https://review.openstack.org/#/c/568999 was rebased care to take a look | |
| 12:38:34 | frickler | coreycb: latest nova stable/pike released switched from ostestr to stestr, which draws a bunch of dependencies in, did someone start working on getting these into pike uca yet? I'd like to avoid duplicating the work | |
| 12:53:33 | coreycb | frickler: i haven't gotten to the latest stable release. we are due for an update for pike. | |
| 12:53:59 | coreycb | frickler: so haven't considered it. but if we need to backport packages that are only used for testing that shouldn't be a problem. | |
| 12:55:27 | sean-k-mooney | coreycb: frickler is this not somthing that should have been done prior to the pike release on ubuntu | |
| 12:55:53 | coreycb | sean-k-mooney: if other projects were using stestr then yes | |
| 12:56:59 | sean-k-mooney | well assuming ubuntu shipped nova it should still have used it but yes i belive several project made the swtich and i think all use it at this point | |
| 12:57:10 | sean-k-mooney | at least the python ones | |
| 12:57:12 | coreycb | sean-k-mooney: it's a little odd to switch dependencies for a stable release though | |
| 12:57:45 | sean-k-mooney | coreycb: yes which is why im questioning backportingt stestr if it was not already done | |