| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-14 | |||
| 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 | |
| 12:58:30 | coreycb | sean-k-mooney: just checked and we don't have stestr backported for pike | |
| 12:58:35 | sean-k-mooney | anyway i gues that up to ye to decided. its a test package so it should be a much lower risk | |
| 12:59:27 | sean-k-mooney | the change to stestr was made in pike right so its not that its not backprot it was never packaged when the release was made | |
| 13:00:14 | sean-k-mooney | or are you saying pike used ostester and frickler was wondering could you backport stestr to use instead on pike | |
| 13:01:20 | coreycb | sean-k-mooney: it appears that no projects in the cloud archive for pike currently use stestr so we may not have packaged it for pike | |
| 13:03:19 | sean-k-mooney | coreycb: so in that case the unit and functional tests would not have been runable for nova on pike without installing stestr from pip | |
| 13:04:09 | coreycb | sean-k-mooney: right, well s/wouldn't/won't/ in our case because we're not at 16.1.6 yet | |
| 13:05:40 | sean-k-mooney | coreycb: well its listed in the test requirements.txt https://github.com/openstack/nova/blob/stable/pike/test-requirements.txt#L20 so i you have the deb installed and you go to the site pacages directory you should just be able to run tox to run the tests and it will install it in the venv | |
| 13:06:13 | sean-k-mooney | that said i guess that would not be testing with the distro packages | |
| 13:06:35 | sean-k-mooney | i belive you can still run the test with ostester so its not a hard requirement | |
| 13:06:46 | coreycb | sean-k-mooney: yeah but that's not how debian packages work | |
| 13:07:35 | coreycb | sean-k-mooney: i'll raise an issue with you all if we have any problems. | |
| 13:07:50 | coreycb | sean-k-mooney: but i think we can probably figure it out | |