| *** spartakos has quit IRC | 00:19 | |
| *** tristanC has quit IRC | 00:33 | |
| *** tristanC has joined #openstack-nova | 00:33 | |
| *** hshiina has joined #openstack-nova | 00:34 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | 00:42 |
|---|---|---|
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | 00:43 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 00:45 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Nova Cyborg interaction specification. https://review.openstack.org/603955 | 00:46 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | 00:51 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106 | 00:51 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110 | 00:51 |
| *** dave-mccowan has quit IRC | 00:52 | |
| *** gyee has quit IRC | 00:54 | |
| *** tetsuro has joined #openstack-nova | 01:05 | |
| *** mhen has quit IRC | 01:05 | |
| *** mhen has joined #openstack-nova | 01:06 | |
| *** mdbooth has quit IRC | 01:16 | |
| *** mdbooth has joined #openstack-nova | 01:16 | |
| *** mdbooth has quit IRC | 01:18 | |
| *** mdbooth has joined #openstack-nova | 01:19 | |
| *** mrsoul has quit IRC | 01:20 | |
| *** mdbooth has quit IRC | 01:22 | |
| *** mdbooth has joined #openstack-nova | 01:22 | |
| *** mdbooth has quit IRC | 01:25 | |
| *** mdbooth has joined #openstack-nova | 01:26 | |
| *** mdbooth has quit IRC | 01:29 | |
| *** bhagyashris has joined #openstack-nova | 01:29 | |
| *** mdbooth has joined #openstack-nova | 01:29 | |
| *** tetsuro has quit IRC | 01:41 | |
| *** tetsuro has joined #openstack-nova | 01:41 | |
| *** tbachman has quit IRC | 01:51 | |
| *** tetsuro has quit IRC | 01:51 | |
| *** takashin has quit IRC | 01:52 | |
| *** tetsuro has joined #openstack-nova | 01:55 | |
| *** hongbin has joined #openstack-nova | 01:55 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:57 | |
| *** tetsuro has quit IRC | 02:01 | |
| *** tetsuro has joined #openstack-nova | 02:03 | |
| *** mhen has quit IRC | 02:03 | |
| *** Dinesh_Bhor has quit IRC | 02:05 | |
| *** mhen has joined #openstack-nova | 02:07 | |
| *** cfriesen has quit IRC | 02:09 | |
| *** tbachman has joined #openstack-nova | 02:15 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:19 | |
| *** Bhujay has joined #openstack-nova | 02:24 | |
| *** takashin has joined #openstack-nova | 02:36 | |
| *** pooja-jadhav has joined #openstack-nova | 02:42 | |
| *** pooja_jadhav has quit IRC | 02:46 | |
| *** jding1_ has joined #openstack-nova | 03:00 | |
| *** ircuser-1 has quit IRC | 03:01 | |
| *** Bhujay has quit IRC | 03:03 | |
| *** jackding has quit IRC | 03:03 | |
| *** ircuser-1 has joined #openstack-nova | 03:06 | |
| openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Nova Cyborg interaction specification. https://review.openstack.org/603955 | 03:09 |
| openstackgerrit | Merged openstack/nova stable/ocata: Fix the help for the disk_weight_multiplier option https://review.openstack.org/607537 | 03:11 |
| openstackgerrit | Merged openstack/nova stable/ocata: Fix host validity check for live-migration https://review.openstack.org/590611 | 03:11 |
| openstackgerrit | Merged openstack/nova stable/queens: stable-only: fix typo in IVS related privsep method https://review.openstack.org/604817 | 03:11 |
| *** udesale has joined #openstack-nova | 03:14 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting migrations https://review.openstack.org/607798 | 03:14 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Follow up "Fix up userdata argument to rebuild" https://review.openstack.org/607800 | 03:22 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting instance action records https://review.openstack.org/607801 | 03:27 |
| *** hongbin has quit IRC | 03:27 | |
| *** Dinesh_Bhor has quit IRC | 03:30 | |
| *** takashin has quit IRC | 03:30 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting migrations https://review.openstack.org/607798 | 03:30 |
| *** sapd1_ has quit IRC | 03:42 | |
| *** sapd1 has joined #openstack-nova | 03:42 | |
| *** takashin has joined #openstack-nova | 03:55 | |
| *** tetsuro has quit IRC | 04:06 | |
| *** pooja-jadhav has quit IRC | 04:14 | |
| *** kukacz has quit IRC | 04:14 | |
| *** kukacz has joined #openstack-nova | 04:16 | |
| *** pooja_jadhav has joined #openstack-nova | 04:16 | |
| pooja_jadhav | Hi team, anyone knows about the api_sample_tests in detail.. I have some doubts in it. | 04:36 |
| *** Dinesh_Bhor has joined #openstack-nova | 04:38 | |
| *** udesale has quit IRC | 05:09 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** tetsuro has joined #openstack-nova | 05:14 | |
| *** udesale has joined #openstack-nova | 05:15 | |
| *** slaweq has quit IRC | 05:16 | |
| gmann | pooja_jadhav: hi, what is doubt | 05:16 |
| *** sridharg has joined #openstack-nova | 05:18 | |
| openstackgerrit | Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751 | 05:39 |
| *** udesale has quit IRC | 05:53 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting migrations https://review.openstack.org/607798 | 05:56 |
| *** Luzi has joined #openstack-nova | 05:57 | |
| *** udesale has joined #openstack-nova | 05:58 | |
| *** ratailor has joined #openstack-nova | 06:06 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting instance action records https://review.openstack.org/607801 | 06:12 |
| *** brinzhang has joined #openstack-nova | 06:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260 | 06:23 |
| pooja_jadhav | gmann: hello, for the simple tenant usage api, if the instance is BFV then local_gb should not consider DISK_GB. that fix I have done. Now I am trying to add functional test for the same. and I am facing issue while creating BFV instance. | 06:24 |
| pooja_jadhav | gmann: In request template i am passing block_device_mappingv2 object. and its trying to cinder from nova and getting Service catelog empty error. Also checked the existing code. they have used stub.out abd CinderFixture and Its not working in my case. | 06:26 |
| pooja_jadhav | *trying to call cinder from nova | 06:29 |
| *** moshele has joined #openstack-nova | 06:30 | |
| brinzhang | When create network (https://developer.openstack.org/api-ref/compute/#create-network), What does mean of "injected" and "multi_host"? | 06:30 |
| pooja_jadhav | it should not consider flavor.DISK_GB | 06:31 |
| *** rcernin has quit IRC | 06:38 | |
| gmann | pooja_jadhav: api_sample_tests does not call to actual service, they are being mocked using fixture. | 06:38 |
| *** moshele has quit IRC | 06:38 | |
| *** rcernin has joined #openstack-nova | 06:38 | |
| gmann | pooja_jadhav: so you need to use cinder Fixture and change fixture if needed - https://github.com/openstack/nova/blob/85b36cd2f82ccd740057c1bee08fc722209604ab/nova/tests/fixtures.py#L1380 | 06:39 |
| *** pcaruana has joined #openstack-nova | 06:40 | |
| gmann | pooja_jadhav: same network test and other cinder test does. | 06:42 |
| gmann | pooja_jadhav: you can write the tempest integration test where all actual services and their behaviour is being tested. but for sample tests you can simply stub the expected things from cinder and check nova expected response/behavior | 06:43 |
| *** dpawlik has joined #openstack-nova | 06:46 | |
| pooja_jadhav | gmann: yes, same point i am able to see that, I need to use Cinder Fixture for the same. Tried same thing which is done at [1]https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/test_block_device_mapping_boot.py#L23-L27 | 06:50 |
| gmann | pooja_jadhav: yeah, any error you are getting? | 06:51 |
| gmann | you have patch up | 06:51 |
| pooja_jadhav | gmann: I have added same stubs in the https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/test_simple_tenant_usage.py and trying to add 4th as BFV instance. | 06:53 |
| *** maciejjozefczyk has joined #openstack-nova | 06:54 | |
| *** sahid has joined #openstack-nova | 06:55 | |
| pooja_jadhav | gmann: here is the error log http://paste.openstack.org/show/731449/ | 06:55 |
| *** sahid has quit IRC | 06:56 | |
| *** slaweq has joined #openstack-nova | 06:57 | |
| *** sahid has joined #openstack-nova | 06:59 | |
| *** ttsiouts has joined #openstack-nova | 06:59 | |
| gmann | pooja_jadhav: this is going on cinder call? did you use Cinder Fixture ? | 07:00 |
| gmann | pooja_jadhav: if you can push your patch in gerrit (WIP though), i can take a look | 07:00 |
| pooja_jadhav | gmann: yes, In setup method, I have added this line : self.useFixture(nova_fixtures.CinderFixture(self)) | 07:01 |
| pooja_jadhav | self.useFixture(nova_fixtures.CinderFixtureNewAttachFlow(self)) | 07:01 |
| gmann | pooja_jadhav: ok, and you are just calling server create with bdmv2 right | 07:01 |
| pooja_jadhav | gmann: yes, right | 07:02 |
| pooja_jadhav | gmann: Also I checked, v2.32 and v2.42, in https://github.com/openstack/nova/blob/master/nova/tests/functional/api_sample_tests/test_servers.py#L189-L196 they are created BFV instances. they are working perfectly fine. Same things trying to use. but not getting success | 07:04 |
| bauzas | good morning nova | 07:04 |
| pooja_jadhav | gmann: How to use this Cinder Fixture?? Am I using correctly in the setup method of the class where I need it ?? | 07:06 |
| gibi | mriedem, efried: I've read back on yesterday's concurrent update. I think one of the goals of the consumer gen patches to make the concurrent update visible and not overwrite allocations so I feel what mriedem saw was a result of that | 07:07 |
| gibi | mriedem, efried: also I agree with efried that if the last patch that updates claim_resources is merged then that parallel allocation request case will stop eariler in the claim_resources when the second claim_resources call for the same consumer fails on the assumption that the consumer does not exists | 07:09 |
| *** rcernin has quit IRC | 07:10 | |
| gmann | pooja_jadhav: i think so. but somehow it is not considering it. | 07:11 |
| gmann | pooja_jadhav can you paste the test_simple_tenant_usage.py or if you can push patch up in gerrit then it is easy to debug | 07:12 |
| *** bhagyashris has quit IRC | 07:16 | |
| *** ttsiouts has quit IRC | 07:16 | |
| pooja_jadhav | gmann: http://paste.openstack.org/show/731450/ | 07:22 |
| *** helenafm has joined #openstack-nova | 07:25 | |
| gmann | pooja_jadhav: you do not need to use both, use CinderFixture only | 07:29 |
| *** udesale has quit IRC | 07:30 | |
| pooja_jadhav | gmann: I have trued by using used CinderFixture only. But still no success L( | 07:31 |
| pooja_jadhav | :( | 07:31 |
| pooja_jadhav | tried* | 07:31 |
| gmann | pooja_jadhav: and the sample file path you changed ? | 07:38 |
| pooja_jadhav | gmann: I am adding sample files inside https://github.com/openstack/nova/tree/85b36cd2f82ccd740057c1bee08fc722209604ab/nova/tests/functional/api_sample_tests/api_samples/os-simple-tenant-usage/v2.40 | 07:39 |
| pooja_jadhav | and passing use_common_server_api_samples=False to _post_server() method so that, It will get the samples from api_samples/os-simple-tenant-usage/v2.40 only. | 07:40 |
| gmann | pooja_jadhav: that's all you are doing perfectly | 07:43 |
| pooja_jadhav | gmann: yes, I will try my best for this. If anything u think I am missing something then please let me know. :) | 07:44 |
| gmann | pooja_jadhav: ok, but if you can push WIP patch up then it will be easy otherwise, it is difficult to debug | 07:46 |
| *** hshiina has quit IRC | 07:47 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | 07:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore forcing of evacuation for nested instance https://review.openstack.org/606111 | 07:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | 07:50 |
| *** udesale has joined #openstack-nova | 07:53 | |
| *** jpena|off is now known as jpena | 07:53 | |
| *** mvkr has quit IRC | 07:55 | |
| *** pcaruana has quit IRC | 07:55 | |
| *** mvkr has joined #openstack-nova | 07:57 | |
| *** pcaruana has joined #openstack-nova | 07:57 | |
| *** ttsiouts has joined #openstack-nova | 08:00 | |
| *** tssurya has joined #openstack-nova | 08:03 | |
| *** Dinesh_Bhor has quit IRC | 08:05 | |
| *** ralonsoh has joined #openstack-nova | 08:06 | |
| *** bhagyashris has joined #openstack-nova | 08:09 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:10 | |
| *** derekh has joined #openstack-nova | 08:13 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update --max-rows parameter description for archive_deleted_rows https://review.openstack.org/606995 | 08:16 |
| *** Dinesh_Bhor has quit IRC | 08:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:26 | |
| *** ralonsoh has quit IRC | 08:30 | |
| *** Dinesh_Bhor has quit IRC | 08:31 | |
| *** ralonsoh has joined #openstack-nova | 08:37 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:38 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Modify get_by_cell_and_project() to get_not_qfd_by_cell_and_project() https://review.openstack.org/607663 | 08:43 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785 | 08:43 |
| *** panda|off is now known as panda | 08:44 | |
| *** udesale has quit IRC | 08:46 | |
| *** ttsiouts has quit IRC | 08:49 | |
| *** udesale has joined #openstack-nova | 08:49 | |
| tssurya | sean-k-mooney: good morning, around ? | 08:55 |
| tssurya | its about the exception handling for the scatter utility stuff we discussed yesterday, would you be modifying your existing patch or should I put a new one for the simple approach ? asking just to rebase the series | 08:58 |
| *** ttsiouts has joined #openstack-nova | 08:59 | |
| *** sapd1 has quit IRC | 09:02 | |
| *** sapd1 has joined #openstack-nova | 09:03 | |
| *** ttsiouts has quit IRC | 09:04 | |
| *** ttsiouts has joined #openstack-nova | 09:08 | |
| gmann | pooja_jadhav: seems like you need to override the nova.volume.cinder.create which is not in conderFixture | 09:08 |
| gmann | t i tested with that and it worked fine | 09:09 |
| *** alexchadin has joined #openstack-nova | 09:10 | |
| *** jangutter has quit IRC | 09:10 | |
| *** alexchadin has quit IRC | 09:10 | |
| gmann | pooja_jadhav: i mean stub | 09:12 |
| tssurya | sean-k-mooney: nvm, my bad just saw http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-10-03.log.html#t2018-10-03T19:17:25 | 09:13 |
| *** jangutter has joined #openstack-nova | 09:14 | |
| sean-k-mooney | am i am actully im just making coffee | 09:16 |
| sean-k-mooney | this is around the time in normally wake up and start being online | 09:16 |
| sean-k-mooney | tssurya: for the simple approch we agreed to just retrun the exception by construction a new copy and leave a comment saying we are doing that because of https://www.python.org/dev/peps/pep-0344/#open-issue-garbage-collection | 09:18 |
| sean-k-mooney | tssurya: so expect execption e: retrun e.class(e.args) | 09:19 |
| tssurya | sean-k-mooney:ack thanks :) sorry for the early ping then | 09:19 |
| tssurya | I was calculating 1hr from behing here | 09:19 |
| tssurya | 1hr behind the time in CEST I mean | 09:20 |
| sean-k-mooney | haha no worries. i shift my day to give me more overlap with the us also im not a morning person :) | 09:20 |
| tssurya | I'll put something up then for the spec's usecase and add you as a reviewer :) | 09:21 |
| tssurya | thanks again | 09:21 |
| *** brinzhang has quit IRC | 09:25 | |
| *** s10 has joined #openstack-nova | 09:26 | |
| sean-k-mooney | bauzas: are you online? | 09:29 |
| bauzas | sean-k-mooney: I am, but I'll need to get my daughters in 10 mins | 09:33 |
| sean-k-mooney | no worries we can chat when you get back its about vgpus | 09:33 |
| bauzas | sean-k-mooney: ok, for what ? | 09:36 |
| sean-k-mooney | i want to know how libvirt/nova detect something is a gpu | 09:36 |
| sean-k-mooney | also i have some good news for you | 09:37 |
| bauzas | sean-k-mooney: we don't really detect whether it's a gpu | 09:37 |
| bauzas | sean-k-mooney: we just say "heh, look, there are some pci devices that support mdevs" | 09:38 |
| sean-k-mooney | bauzas: mnaser: is amazing and is going to provide 1-2 vm instance with gpus for testing in the gate https://review.openstack.org/#/c/607686/ | 09:38 |
| bauzas | cool | 09:38 |
| bauzas | so I could write a functional test | 09:38 |
| sean-k-mooney | bauzas: ok that is a bug but also cool because i think i konw how to fake them in the gate also | 09:38 |
| bauzas | sean-k-mooney: what's a bug ? | 09:38 |
| bauzas | to say a pci device supporting mdevs is a gpu ? | 09:39 |
| sean-k-mooney | nic and acclerator can expose mdevs | 09:39 |
| bauzas | sure, but we have a spec fixing this | 09:39 |
| bauzas | sean-k-mooney: in this spec, you explicitly say which PCI IDs *are* GPUs | 09:40 |
| bauzas | sean-k-mooney: https://review.openstack.org/#/c/602474/ | 09:40 |
| bauzas | I need to go, bbiabn | 09:40 |
| sean-k-mooney | bauzas: cool that will work for the terible hack i want to do. ill let you know when you get back | 09:40 |
| *** Bhujay has joined #openstack-nova | 09:48 | |
| *** priteau has joined #openstack-nova | 09:49 | |
| *** bhagyashris has quit IRC | 09:50 | |
| gmann | pooja_jadhav: hope that worked. | 09:51 |
| *** mdbooth has joined #openstack-nova | 09:51 | |
| *** alexchadin has joined #openstack-nova | 09:53 | |
| *** dtantsur|afk is now known as dtantsur | 09:55 | |
| *** mdbooth_ has joined #openstack-nova | 09:57 | |
| *** moshele has joined #openstack-nova | 09:59 | |
| *** mdbooth has quit IRC | 09:59 | |
| *** alexchadin has quit IRC | 10:02 | |
| *** udesale has quit IRC | 10:04 | |
| *** mdbooth has joined #openstack-nova | 10:04 | |
| *** alexchadin has joined #openstack-nova | 10:05 | |
| *** Bhujay has quit IRC | 10:06 | |
| *** mdbooth_ has quit IRC | 10:07 | |
| *** mdbooth_ has joined #openstack-nova | 10:07 | |
| *** Dinesh_Bhor has quit IRC | 10:08 | |
| *** mdbooth has quit IRC | 10:10 | |
| bauzas | sean-k-mooney: just able to discuss for 5 mins | 10:12 |
| bauzas | sean-k-mooney: but also, without the spec, it also only works for GPU devices | 10:12 |
| bauzas | sean-k-mooney: because libvirt provides us all the mdev capable devices, but we look at which supports the provided GPU type | 10:12 |
| bauzas | say 'nvidia-11' | 10:12 |
| bauzas | so, in case you have mdev capable devices that are not GPU devices (say a nic one), they don't support the type | 10:13 |
| *** mdbooth has joined #openstack-nova | 10:13 | |
| bauzas | so we don't use them | 10:13 |
| *** udesale has joined #openstack-nova | 10:13 | |
| bauzas | also, say you have an Intel GPU not supporting the 'nvidia-11' type, then we don't have inventory for it too | 10:14 |
| sean-k-mooney | ok so your assumeing good nameing of the mdev types without the spec | 10:15 |
| sean-k-mooney | e.g. not small medium large | 10:15 |
| *** mdbooth_ has quit IRC | 10:15 | |
| sean-k-mooney | but ya i think that is fine | 10:15 |
| sean-k-mooney | basically i found out that in kernel 4.16 there is a new driver called netdevsim that simulates a nic | 10:15 |
| sean-k-mooney | it supprots sriov without any hardware and mdevs | 10:16 |
| sean-k-mooney | so i think i can use it to test pci passhtrouhg, sriov, and other things in the gate | 10:16 |
| sean-k-mooney | i also may be able to use the mdev support to pretend to be a vgpu | 10:17 |
| *** mdbooth_ has joined #openstack-nova | 10:17 | |
| sean-k-mooney | not sure about the last point but i was planning to look into it more for sriov testing | 10:17 |
| *** mdbooth has quit IRC | 10:17 | |
| *** alexchadin has quit IRC | 10:19 | |
| *** ttsiouts has quit IRC | 10:21 | |
| *** Dinesh_Bhor has joined #openstack-nova | 10:21 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Move the evacuate action to admin action https://review.openstack.org/607896 | 10:22 |
| *** ttsiouts has joined #openstack-nova | 10:31 | |
| *** slaweq_ has joined #openstack-nova | 10:38 | |
| *** udesale has quit IRC | 10:39 | |
| *** alexchadin has joined #openstack-nova | 10:39 | |
| *** udesale has joined #openstack-nova | 10:40 | |
| *** slaweq has quit IRC | 10:40 | |
| *** tbachman has quit IRC | 10:44 | |
| *** erlon has joined #openstack-nova | 10:49 | |
| *** slaweq__ has joined #openstack-nova | 10:56 | |
| *** slaweq_ has quit IRC | 10:57 | |
| *** pcaruana has quit IRC | 11:02 | |
| *** pcaruana has joined #openstack-nova | 11:02 | |
| *** vivsoni has quit IRC | 11:03 | |
| *** jpena is now known as jpena|lunch | 11:03 | |
| *** janki has joined #openstack-nova | 11:06 | |
| *** mdbooth_ has quit IRC | 11:13 | |
| *** cdent has joined #openstack-nova | 11:14 | |
| *** mdbooth has joined #openstack-nova | 11:14 | |
| *** Dinesh_Bhor has quit IRC | 11:17 | |
| *** alexchadin has quit IRC | 11:24 | |
| *** ttsiouts has quit IRC | 11:32 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: consumer gen: support claim_resources https://review.openstack.org/583667 | 11:46 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672 | 11:46 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use provider tree in virt FakeDriver https://review.openstack.org/604083 | 11:46 |
| gibi | jaypipes, efried: I replied in https://review.openstack.org/583667 for both of you fixed your comments. The runway slot for this patch ends today so I appreciate any feedback | 11:49 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Refactor scatter-gather utility to return exception objects https://review.openstack.org/607934 | 11:49 |
| *** slagle has quit IRC | 11:51 | |
| jaypipes | gibi: yup, on it. | 11:52 |
| gibi | jaypipes: thanks a lot | 11:52 |
| bauzas | sean-k-mooney: back there | 11:54 |
| bauzas | sean-k-mooney: if you wanna continue to discuss... | 11:54 |
| bauzas | gibi: don't worry about the runway slot, now that I'm done with the reshaper change, your series is my top prio | 11:54 |
| gibi | bauzas: cool, thanks | 11:55 |
| bauzas | gibi: so, the above means that I can review it, right? | 11:55 |
| gibi | bauzas: it means that tha last patch of consumer gen is updated with fixes based on jaypipes and efried comments, there are patches top of that that also up to date til https://review.openstack.org/604083 I'm fixing comment in the rest as we speak | 11:56 |
| bauzas | gibi: okay, will look later today | 11:56 |
| bauzas | gibi: FWIW I have a machine with GPU devices so I can test most of your series | 11:57 |
| gibi | bauzas: sounds cool :) | 11:57 |
| gibi | bauzas: so far I relied on functional test cases. Fortunately the resource allocation handling these patches change are possible to cover with functional | 11:58 |
| gibi | bauzas: the physical consumpton of VGPUs are something where my functional test case wont be enough | 11:59 |
| sean-k-mooney | jaypipes: bauzas gibi so in case you missed it earlier i think i figured out a way to test pci passthough, sriov, and some other things like livemigration with the same in the gate using the netdevsim kernel module | 11:59 |
| gibi | sean-k-mooney: that is really good news! | 11:59 |
| bauzas | gibi: I mean, I could just test your series with my reshape change | 11:59 |
| bauzas | gibi: so that I could be sure we would be able to boot a flavor asking for VGPUs | 12:00 |
| bauzas | (in a nested worldà | 12:00 |
| sean-k-mooney | i also think that can be hacked to pretend to be vgpus i need to investgate that futher | 12:00 |
| gibi | bauzas: yeah allocate, then rehape, then allocate again is something that needs coverage | 12:00 |
| bauzas | gibi: I tested all of that with my single change | 12:00 |
| gibi | bauzas: cool | 12:00 |
| bauzas | gibi: but of course the boot failed | 12:00 |
| bauzas | gibi: since I wasn't using your series | 12:01 |
| bauzas | what I could do is just to pick your series, and do the same | 12:01 |
| gibi | bauzas: I assume it will work with my series included | 12:01 |
| bauzas | gibi: that's my assumption too | 12:01 |
| bauzas | and the functional test shows it | 12:01 |
| bauzas | but a real evidence is better, I guess | 12:01 |
| gibi | bauzas: totally agree | 12:01 |
| bauzas | sean-k-mooney: WDYM by netdevsim module ? | 12:02 |
| sean-k-mooney | it is a kernel module adding in kernel 4.16 specifcally to allow testing of hardware offloads without hardwar | 12:02 |
| jaypipes | sean-k-mooney: you'll excuse my skepticism until I see a real functional test of those items :) | 12:02 |
| sean-k-mooney | bauzas: it allows you to create pci device that support mdevs and sriov | 12:03 |
| bauzas | sean-k-mooney: but it requires 4.16, right? | 12:03 |
| sean-k-mooney | and it support ebpf too | 12:03 |
| sean-k-mooney | yep so a fedora 28 job | 12:03 |
| bauzas | which could be a problem | 12:03 |
| jaypipes | sean-k-mooney: that said, I'm still supportive of this netdevsim effort. ANYTHING is better than what we have now, which is pretty much nothing. | 12:03 |
| sean-k-mooney | fedora comes with 4.17 | 12:03 |
| *** jpena|lunch is now known as jpena | 12:04 | |
| *** tetsuro has quit IRC | 12:04 | |
| bauzas | sean-k-mooney: but bionic comes with 4.15 https://packages.ubuntu.com/bionic/linux-image-generic | 12:04 |
| bauzas | so it wouldn't be a classic upstream gate job | 12:04 |
| sean-k-mooney | jaypipes: ya i just learned about it yesterday so still doing reasarch but since i plan on working on sirov livemigration this cycle i would like to try to create an experimental job with this moduel to test it | 12:05 |
| bauzas | sean-k-mooney: I was also considering the use of fake libvirt | 12:05 |
| sean-k-mooney | bauzas: we have fedroa 28 images avaliable in the upstream gate for testing | 12:05 |
| bauzas | couldn't that be simplier ? | 12:05 |
| sean-k-mooney | bauzas: it could be this would allow tempest testing not just functional | 12:05 |
| bauzas | sean-k-mooney: for SR-IOV it could be interesting | 12:06 |
| bauzas | sean-k-mooney: for VGPU, I just feel it's unnecessary thru tempest | 12:06 |
| bauzas | a functional test in-tree is better IMO | 12:06 |
| sean-k-mooney | yep it also support things like configring rate limiting on the vf and trusted vfs | 12:07 |
| bauzas | sean-k-mooney: just to make it clear, I was considering use of https://libvirt.org/drvtest.html | 12:07 |
| *** ttsiouts has joined #openstack-nova | 12:07 | |
| *** vivsoni has joined #openstack-nova | 12:07 | |
| *** med_ has joined #openstack-nova | 12:08 | |
| sean-k-mooney | ok but i doubt that will support the featue we want to test | 12:08 |
| sean-k-mooney | on testing with real hardware there is also this change https://review.openstack.org/#/c/607686/ :) | 12:09 |
| jaypipes | sean-k-mooney: isn't artom working on that as well? (sriov live migration...) | 12:09 |
| jaypipes | sean-k-mooney: or is artom focused on NUMA stuffs? | 12:09 |
| sean-k-mooney | jaypipes: artom is working on numa | 12:09 |
| jaypipes | ah, gotcha | 12:09 |
| sean-k-mooney | im goning to do the sriov part | 12:09 |
| jaypipes | sean-k-mooney: you're both brave men. | 12:10 |
| sean-k-mooney | or dumb | 12:10 |
| jaypipes | sean-k-mooney: I wouldn't touch that stuff with a ten foot pole. | 12:10 |
| *** erlon has quit IRC | 12:10 | |
| jaypipes | sean-k-mooney: the closest I'll get is doing some reviews for ya ;) | 12:10 |
| sean-k-mooney | well i keep finding bugs in livemigration that i have to fix first related to the multiple port bindings on both the nova and neutron sides | 12:11 |
| *** mdbooth has quit IRC | 12:11 | |
| sean-k-mooney | so first im going to try and fix all those bugs then sriov live migration. i think neutron forgot to update teh sriovnic agent and driver to support multiple port binding so i have to fix that first | 12:12 |
| artom | jaypipes, since you mentioned it, review pretty please https://review.openstack.org/#/c/599587/ ? ;) | 12:13 |
| artom | But yeah, there's going to be some overlap/cooperation | 12:14 |
| artom | Also, the mellanox guys were interested in SRIOV live migration, not sure what happened with that | 12:14 |
| sean-k-mooney | artom: i talked to moshele last week. ill try and work with them if they want to help. | 12:14 |
| artom | sean-k-mooney, ah, cool | 12:15 |
| sean-k-mooney | artom: moshele said someone on his team was working on a poc | 12:15 |
| artom | Hrmm, I thought they were going to propose a spec first? | 12:15 |
| sean-k-mooney | well i already wrote one | 12:15 |
| sean-k-mooney | artom: https://review.openstack.org/#/c/605116/ | 12:15 |
| artom | sean-k-mooney, thanks, was about to ask | 12:16 |
| *** tbachman has joined #openstack-nova | 12:17 | |
| sean-k-mooney | jaypipes: bauzas gibi sorry your conversation got a little derailed by that but i think using the new vexhost physical gpu nodes in an experimental job would also be very valueable for testing both reshaper stuff and vgpus | 12:19 |
| jaypipes | sean-k-mooney: but that would require us having to talk to mnaser and we ALL know that's not a good idea! | 12:20 |
| jaypipes | artom: queued that up behind the gibster. | 12:21 |
| jaypipes | gibi: OK, so... question on your response about the microversion 1.29 thing in https://review.openstack.org/#/c/583667/25/nova/scheduler/client/report.py... | 12:21 |
| sean-k-mooney | jaypipes: :) well you could ignore him and just submit jobs when https://review.openstack.org/#/c/607686/ is merged | 12:22 |
| gibi | jaypipes: looking.. | 12:23 |
| jaypipes | gibi: what is API 1.29 changing about GET /a_c's return? | 12:23 |
| jaypipes | gibi: you mention this: "So by supporting nested a_c we implicitly force nova to at least support 1.29 >= in claim_resources too." | 12:23 |
| jaypipes | gibi: but I'm wondering what changes in the allocation_request part of the a_c response to warrant a change in this code. | 12:23 |
| sean-k-mooney | jaypipes: isnt 1.29 the microverion that adds nested allocation candiates? | 12:24 |
| jaypipes | sean-k-mooney: but it's not "nested allocation candidates" really... the allocation request part of the response is still just a flat list of providers and the resource amounts being consumed from each. | 12:24 |
| jaypipes | or at least, that's what I thought... | 12:25 |
| sean-k-mooney | jaypipes: you are proably right i just have a vague recolection from the demo at the ptg that there was a reson this microverion was need for nested allocations | 12:25 |
| *** mrch_ has joined #openstack-nova | 12:26 | |
| gibi | jaypipes: technicall the allocations structure is unchanged in 1.29 but handling the fact that now more than one RP can be in an allocation candidate needs code change in multiple places in nova. Some of them is trivially missed in first patch that enables nested a_c in nova hence the expectedFailures in the functional test | 12:27 |
| *** mdbooth has joined #openstack-nova | 12:27 | |
| gibi | jaypipes: for example nova assumes that deleting an instance allocation from a compute is as easy as deleting the allocation from the compute RP | 12:28 |
| sean-k-mooney | gibi: if the allocation candiates are a flat list as jaypipes says above is the provider topology captured in the summery? | 12:28 |
| gibi | sean-k-mooney: yes, parent_rp_uuid is in the summary part | 12:29 |
| gibi | s/parent_rp_uuid/parent_provider_uuid | 12:29 |
| jaypipes | sean-k-mooney: yeah, it's in the provider_summaries part of the response, not the allocation_requests part of the response, which is what I was alluding to above. | 12:29 |
| *** erlon has joined #openstack-nova | 12:29 | |
| sean-k-mooney | so in the delete case nova now needs to delete the allocation from all resouce providers in the list instead of just one delete | 12:29 |
| jaypipes | sean-k-mooney, gibi: and we don't pass the provider_summaries response to the claim_resources() method (only the allocation_request part) which is why I was asking about that comment from gibi on the claim_resources() patch. | 12:30 |
| *** mdbooth_ has joined #openstack-nova | 12:30 | |
| gibi | sean-k-mooney: exactly. It is implemented in https://review.openstack.org/#/c/606050/ | 12:30 |
| sean-k-mooney | i assume there is no api to say delete all allocation for this consumer uuid? | 12:30 |
| jaypipes | sean-k-mooney: there is, yes. | 12:30 |
| cdent | 2 even | 12:30 |
| jaypipes | cdent: touche :) | 12:31 |
| sean-k-mooney | so in that case for a delete cant nova just do that and pass the instance uuid? | 12:31 |
| gibi | sean-k-mooney: it is complicated if the consumer has allocations on other computes as well | 12:31 |
| gibi | sean-k-mooney: and it is the case for evacuate :/ | 12:31 |
| jaypipes | sean-k-mooney: sure it can. the issue is edge cases... gibi's func test patch outlines those cases well. lemme grab you a link. | 12:31 |
| gibi | jaypipes: I think I'm failing to grasp what is exactly your suggestion for claim_resources() call | 12:32 |
| mrch_ | https://bugs.launchpad.net/nova/+bug/1763043 ( Instance not resizing, skipping migration.) well its not unnecessary because 70% of them have locked nova/cinder disk, got around a dozend of them any ideas, excetp "rbd lock remove" till the end of my life? | 12:32 |
| openstack | Launchpad bug 1763043 in OpenStack Compute (nova) "Unnecessary "Instance not resizing, skipping migration" warning in n-cpu logs during live migration" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | 12:32 |
| sean-k-mooney | gibi: for evacuate would we not use a migration uuid to hold the dest allocations and then not delete the source allocation and swap it over like we do for cold migrate? | 12:32 |
| gibi | sean-k-mooney: that would be ideal, but does not happen today | 12:33 |
| jaypipes | sean-k-mooney: see very bottom of this file: https://review.openstack.org/#/c/604084/3/nova/tests/functional/test_servers.py | 12:33 |
| sean-k-mooney | im really not familar enough with this code unfortuenetly | 12:33 |
| gibi | sean-k-mooney: I have a todo from the PTG to improve that as well | 12:33 |
| *** mdbooth has quit IRC | 12:33 | |
| jaypipes | sean-k-mooney: those tests and comments from gibi highlight well the issue. | 12:33 |
| jaypipes | issues... | 12:33 |
| * jaypipes shakes fist at the Gods of Evacuate. | 12:34 | |
| bauzas | jaypipes: flush the toilets | 12:35 |
| sean-k-mooney | jaypipes i have no doubt gibi has reasoned about this and the edgecase far better then i :) espcially since i jsut stared looking at the patch but ya just providing my assumtions in case that help with any that might have been made :) my assumetion of how this should work likely diverge hevily form how it does | 12:35 |
| gibi | jaypipes: there is stuctural change between 1.12-1.28 but there is no strucutral change when we step from 1.28 to 1.29 in a_c but I don't know what you want to suggest | 12:36 |
| jaypipes | gibi: I was questioning the need to "pin" the claim_resources() API microversion to 1.29 since there was no change to the allocation_request part of the a_c response. | 12:37 |
| gibi | jaypipes: sure claim_resources can work with 1.28 even if a_c used 1.29. I think we are not even bumping claim_resources to 1.29 when we bump a_c to 1.29 | 12:39 |
| gibi | jaypipes: this path bumps a_c to 1.29 https://review.openstack.org/#/c/585672 but not the claim_resources | 12:39 |
| jaypipes | gibi: k, cool. | 12:41 |
| jaypipes | gibi: again, I was just responding to your comments on https://review.openstack.org/#/c/583667/25/nova/scheduler/client/report.py where you said this: | 12:41 |
| jaypipes | "Therefore scheduler passes down >= 1.29 when calling claim_resources with the selected allocation candidate. So by supporting nested a_c we implicitly force nova to at least support 1.29 >= in claim_resources too. " | 12:41 |
| jaypipes | gibi: but, no worries, I'll stop bothering you and get back to reviewing the last revision in that patch :) | 12:42 |
| gibi | jaypipes: so you suggets to drop the logic that passes version from a_c to claim_resources? | 12:42 |
| jaypipes | gibi: yeah, might be useful... | 12:42 |
| gibi | jaypipes: that logic is the one that forces claim_resources to adapt to 1.29 | 12:42 |
| jaypipes | gibi: since technically there's nothing about 1.29 that affects claim_resources. | 12:43 |
| jaypipes | gibi: and the allocation_request_version that gets passed to claim_resources() is all about the structure of the allocation_request part of the payload. | 12:43 |
| jaypipes | gibi: but it's really not a huge deal :) | 12:43 |
| gibi | jaypipes: jeah for 1.29 we don't need that but for the bump of 1.12 - 1.28 we need to know that the a_c was used with 1.28 | 12:44 |
| jaypipes | gibi: like I said, I'll go back to my last revision review :) | 12:44 |
| gibi | jaypipes: I let you review :) | 12:44 |
| jaypipes | gibi: elnézést a zavarásért! | 12:44 |
| * jaypipes hopes that is even remotely correct... | 12:45 | |
| jaypipes | I imagine gibi laughing at Google Translate's poor translation | 12:45 |
| gibi | jaypipes: what was the English expression you translated? | 12:45 |
| gibi | :) | 12:46 |
| jaypipes | "Sorry to bother" | 12:46 |
| jaypipes | lol | 12:46 |
| jaypipes | what does it mean? :) | 12:46 |
| jaypipes | "Happy cows love coding" or something wacky like that? | 12:46 |
| *** slagle has joined #openstack-nova | 12:47 | |
| gibi | no it is not that far | 12:47 |
| * gibi is now strugling to define the difference he feels | 12:48 | |
| gibi | jaypipes: what you used is more for the case when you want to approach a stranger, e.g. as a first sentence before you ask for direction | 12:50 |
| gibi | so it is super polite | 12:51 |
| gibi | but yeah you basically asked for forgiveness for the interruption | 12:51 |
| gibi | so the intention is clear | 12:52 |
| * gibi wonders if google improved the hungarian translation a lot lately | 12:52 | |
| jaypipes | hehe | 12:54 |
| *** moshele has quit IRC | 12:56 | |
| *** med_ has quit IRC | 12:57 | |
| bauzas | jaypipes: gibi: https://www.duolingo.com/course/hu/en/ | 12:58 |
| bauzas | I did that for German \o/ | 12:58 |
| bauzas | und jetzt I spreche Deutch | 12:59 |
| bauzas | Ich* | 12:59 |
| sean-k-mooney | bauzas: if you ever want to lean a languge that no one will under stand they have an irish course too | 13:00 |
| *** mvkr has quit IRC | 13:02 | |
| bauzas | sean-k-mooney: I saw this | 13:03 |
| bauzas | there is also High Valeryan and Klingon | 13:03 |
| sean-k-mooney | bauzas: both are proably more useful then irish that said it was nice to have a language you could use to gossip about people abroad on school trips lol only use of irish i have ever had | 13:06 |
| efried | jetzt spreche ich <== with an adjective in the mix, you put the verb before the subject. Or something. | 13:07 |
| efried | or whatever "jetzt" is, not an adjective | 13:07 |
| *** ratailor has quit IRC | 13:21 | |
| *** slagle has quit IRC | 13:22 | |
| jaypipes | gibi: +2 from me on claim_resources(). I will let efried or mriedem +W. | 13:22 |
| gibi | jaypipes: thanks a lot. efried had a bunch of comments so I feel he is the one who can +W it | 13:22 |
| *** erlon_ has joined #openstack-nova | 13:25 | |
| *** tetsuro has joined #openstack-nova | 13:25 | |
| *** erlon has quit IRC | 13:29 | |
| *** mvkr has joined #openstack-nova | 13:30 | |
| *** spatel has joined #openstack-nova | 13:30 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor allocation checking in functional tests https://review.openstack.org/607287 | 13:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run ServerMovingTests with nested resources https://review.openstack.org/604084 | 13:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore forcing of live migration for nested instance https://review.openstack.org/605785 | 13:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | 13:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore forcing of evacuation for nested instance https://review.openstack.org/606111 | 13:31 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | 13:31 |
| gibi | bauzas: FYI, the nested series now up to date all the way to the top https://review.openstack.org/#/c/604125/ | 13:33 |
| *** slaweq__ is now known as slaweq | 13:36 | |
| *** slagle has joined #openstack-nova | 13:41 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Refactor scatter-gather utility to return exception objects https://review.openstack.org/607934 | 13:43 |
| *** eharney has joined #openstack-nova | 13:43 | |
| *** mchlumsky has joined #openstack-nova | 13:45 | |
| *** spatel has quit IRC | 13:47 | |
| *** dpawlik has quit IRC | 13:47 | |
| *** pcaruana has quit IRC | 13:50 | |
| openstackgerrit | Jack Ding proposed openstack/nova master: Handle missing marker during online data migration https://review.openstack.org/605164 | 13:50 |
| *** oanson has quit IRC | 13:51 | |
| *** alexchadin has joined #openstack-nova | 13:56 | |
| *** alexchadin has quit IRC | 13:56 | |
| *** munimeha1 has joined #openstack-nova | 13:58 | |
| *** tbachman has quit IRC | 13:58 | |
| *** alexchadin has joined #openstack-nova | 13:58 | |
| *** tbachman_ has joined #openstack-nova | 13:58 | |
| *** tbachman_ has quit IRC | 13:58 | |
| *** priteau has quit IRC | 13:58 | |
| *** priteau has joined #openstack-nova | 13:59 | |
| dansmith | did I miss that the meeting this week was canceled or something? | 14:02 |
| gibi | dansmith: I think I did not get a request from melwitt to chair it but I can | 14:03 |
| dansmith | meh | 14:03 |
| gmann | not sure. I was also checking meeting channel | 14:03 |
| efried | we have an important topic to discuss, HPET for libvirt. | 14:03 |
| bauzas | hah, I just asked the question in -meeting | 14:03 |
| efried | they're expecting it. | 14:03 |
| bauzas | efried: and what about HCATTLE ? | 14:03 |
| efried | oy vay | 14:03 |
| sean-k-mooney | efried: so #openstack-meeting time i guess | 14:04 |
| efried | yup | 14:04 |
| *** munimeha1 has quit IRC | 14:05 | |
| *** awaugama has joined #openstack-nova | 14:06 | |
| *** hamzy has quit IRC | 14:10 | |
| mrch_ | can i prevent the nova.compute.resource_tracker spaming this: nova.compute.resource_tracker [req-e23cf459-506f-42d3-ac5d-73d87b1e5ba9 - - - - -] [instance: 8ecbbdb1-bad7-434b-8d09-265964ca9c7e] Instance not resizing, skipping migration. when 5 hosts spam this about one an the same vm, an i got 10 vms of that kind, its annoying as f | 14:11 |
| efried | mrch_: We're in the nova meeting atm; can you ask again at the top of the hour? | 14:12 |
| efried | It looks like this is happening because you have some stale migrations in the database; can you clean those up? | 14:13 |
| *** awaugama has quit IRC | 14:14 | |
| *** awaugama has joined #openstack-nova | 14:15 | |
| *** mlavalle has joined #openstack-nova | 14:15 | |
| mrch_ | efried | 14:15 |
| mrch_ | efried: shure | 14:15 |
| *** jaosorior has quit IRC | 14:16 | |
| *** tbachman has joined #openstack-nova | 14:16 | |
| mrch_ | efried | 14:18 |
| mrch_ | efried: concidering db, but i cant wait, i stay in chat just write ill see it tomorrow | 14:18 |
| efried | mrch_: Okay. I personally don't know how to clean up the db, but hopefully one of the wizards can advise once the meeting is over. | 14:18 |
| *** munimeha1_ has joined #openstack-nova | 14:21 | |
| *** mdbooth_ is now known as mdbooth | 14:23 | |
| *** Bhujay has joined #openstack-nova | 14:28 | |
| *** s10_ has joined #openstack-nova | 14:28 | |
| *** spatel has joined #openstack-nova | 14:28 | |
| *** cfriesen has joined #openstack-nova | 14:30 | |
| *** s10 has quit IRC | 14:30 | |
| *** pooja_jadhav has quit IRC | 14:30 | |
| *** kukacz has quit IRC | 14:30 | |
| *** kukacz has joined #openstack-nova | 14:30 | |
| *** pooja_jadhav has joined #openstack-nova | 14:31 | |
| *** liuyulong has joined #openstack-nova | 14:34 | |
| *** hamzy has joined #openstack-nova | 14:45 | |
| *** maciejjozefczyk has quit IRC | 14:46 | |
| *** med_ has joined #openstack-nova | 14:48 | |
| *** itlinux has quit IRC | 14:48 | |
| *** jchhatbar has joined #openstack-nova | 14:50 | |
| *** cdent has quit IRC | 14:53 | |
| *** janki has quit IRC | 14:53 | |
| *** med_ has quit IRC | 14:54 | |
| melwitt | gibi: thank you for running the nova meeting, it completely slipped my mind this time. apologies | 14:58 |
| gibi | melwitt: no worries | 15:00 |
| *** Swami has joined #openstack-nova | 15:00 | |
| mhen | o/ | 15:00 |
| * mhen would like to briefly discuss a topic since time in meeting ran out | 15:00 | |
| mrch_ | can i prevent the nova.compute.resource_tracker spaming this: nova.compute.resource_tracker [req-e23cf459-506f-42d3-ac5d-73d87b1e5ba9 - - - - -] [instance: 8ecbbdb1-bad7-434b-8d09-265964ca9c7e] Instance not resizing, skipping migration. when 5 hosts spam this about one an the same vm, an i got 10 vms of that kind, its annoying as f | 15:01 |
| *** takashin has left #openstack-nova | 15:01 | |
| mhen | as already mentioned in the ML we want to propose the introduction of image encryption in OpenStack | 15:02 |
| mhen | #link ML thread on proposal for image encryption http://lists.openstack.org/pipermail/openstack-dev/2018-September/135167.html | 15:02 |
| mhen | as advised on the ML, we are currently writing individual specs for the components involved, including Nova | 15:02 |
| mhen | in Nova we would like to add the decryption of images, which are converted into servers using ephemeral storage and the encryption of images created from such servers | 15:03 |
| mhen | at first we want to focus on LibVirt and its LVM backend, because it is the only backend for encrypted ephemeral storage in LibVirt (ignoring the ineffectiveness of the encryption due to missing native LUKS support for now) | 15:03 |
| *** cdent has joined #openstack-nova | 15:03 | |
| mhen | we would be interested in Nova's opinion on whether the conversion from encrypted image to unencrypted ephemeral storage should be allowed | 15:03 |
| mhen | we don't really like the idea of exposing the image data this way but the Cinder team suggested making this optional for the similarly natured volume use case | 15:03 |
| efried | jaypipes: Because then the libvirt driver would have to use that (HPET=required) as its cue to turn HPET on in the guest. I.e. using a trait to effect configuration. | 15:04 |
| efried | jaypipes: I'm writing up the alternatives with their respective cons; stay tuned. | 15:05 |
| cfriesen | sean-k-mooney: if we ever wanted to support passthrough of host TPM, then there really would be a finite inventory | 15:05 |
| cfriesen | sean-k-mooney: for emulated TPM we could make it a trait if the host supports it | 15:06 |
| cfriesen | sean-k-mooney: if we ever wanted to support both emulated and passthrough TPM, what would the placement side look like? would that be two resource providers, one with the "emulated" trait and one with the "real" trait? | 15:08 |
| jaypipes | mrch_: why would 5 hosts be spamming that same message? does that mean you have the same instance stuck in a migration to multiple hosts? | 15:09 |
| jaypipes | cfriesen: and that's the problem with embedding key/value things into a trait string... | 15:09 |
| *** s10_ has quit IRC | 15:09 | |
| cfriesen | jaypipes: maybe no traits are needed....in a sense this would be equivalent to VCPU vs PCPU | 15:10 |
| cfriesen | so maybe it's just two separate resource providers | 15:11 |
| *** mrch_ has quit IRC | 15:11 | |
| *** ttsiouts has quit IRC | 15:11 | |
| *** mrch_ has joined #openstack-nova | 15:13 | |
| jaypipes | cfriesen: ack | 15:13 |
| *** mrch_ has quit IRC | 15:20 | |
| *** jding1_ is now known as jackding | 15:23 | |
| spatel | Quick question, i am creating SR-IOV instance and in order to do that i need to create neutron port first. i have created bunch of port during testing but now i need to find out which neutron port is active and non-active how do i find that out? | 15:23 |
| *** mdbooth has quit IRC | 15:24 | |
| spatel | sean-k-mooney: morning | 15:24 |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: WIP: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 15:24 |
| *** mdbooth has joined #openstack-nova | 15:24 | |
| efried | sean-k-mooney, cdent, jaypipes, cfriesen, dansmith, bauzas, gibi: ^ | 15:24 |
| cdent | noted | 15:24 |
| *** Luzi has quit IRC | 15:25 | |
| *** slagle has quit IRC | 15:25 | |
| *** macza has joined #openstack-nova | 15:25 | |
| dansmith | jaypipes: you around for a hangout in 35 minutes on this hpet thing? | 15:25 |
| *** mdbooth has quit IRC | 15:27 | |
| *** mdbooth has joined #openstack-nova | 15:28 | |
| *** mdbooth has quit IRC | 15:28 | |
| *** mdbooth has joined #openstack-nova | 15:29 | |
| *** mdbooth has quit IRC | 15:32 | |
| *** mdbooth_ has joined #openstack-nova | 15:32 | |
| cfriesen | efried: got it. thanks for writing this up | 15:32 |
| jaypipes | dansmith: unfortunately, no. I would be available in about 1.75 hours though | 15:36 |
| *** udesale has quit IRC | 15:36 | |
| dansmith | efried: cfriesen cdent bauzas gibi ^ ? | 15:36 |
| bauzas | sorry I'm trapped in a meeting, trying to catch up | 15:37 |
| bauzas | ah | 15:37 |
| bauzas | I'm fine with meeting us later | 15:37 |
| efried | I can't make it then :( | 15:37 |
| bauzas | the best for me is after 8pm UTC | 15:37 |
| cdent | dansmith: I will have to pass, but I trust the combo of people involved to work out something sane, especially since you were able to identify what is sacred to me (and remains so)( | 15:37 |
| dansmith | heh okay | 15:37 |
| efried | 8pm UTC would wfm | 15:38 |
| jaypipes | that works for me too | 15:38 |
| cfriesen | so a little over 4hrs from now? | 15:38 |
| efried | yes | 15:39 |
| cfriesen | works for me | 15:39 |
| cfriesen | I've never done one of the hangouts....does someone post a URL or send out invites or what? | 15:40 |
| dansmith | url | 15:40 |
| jackding | I will have to pass. | 15:41 |
| efried | cfriesen: dansmith will drop a URL in here, ping all of us, and get spanked by Sigyn | 15:41 |
| cfriesen | I'll cover for jack | 15:41 |
| *** hamzy_ has joined #openstack-nova | 15:42 | |
| *** hamzy has quit IRC | 15:44 | |
| *** mdbooth_ has quit IRC | 15:46 | |
| *** jchhatbar has quit IRC | 15:46 | |
| *** liuyulong has quit IRC | 15:51 | |
| *** Bhujay has quit IRC | 15:55 | |
| *** itlinux has joined #openstack-nova | 15:57 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova show when a cell is down https://review.openstack.org/591658 | 15:58 |
| *** gyee has joined #openstack-nova | 16:00 | |
| efried | gibi: You still around? | 16:02 |
| gibi | efried: yeah for about 10 minutes | 16:02 |
| gibi | efried: but go ahead | 16:02 |
| efried | okay, that may not be enough. I wanted to walk through the allocation_request_version nonsense a bit. | 16:03 |
| *** helenafm has quit IRC | 16:03 | |
| gibi | efried: even if we figure out that we want to drop allocation_request_version I would like to do that separately as that a surgery again | 16:03 |
| efried | it's just, I see it as being pretty much unnecessary/unrelated to this patch. | 16:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore forcing of live migration for nested instance https://review.openstack.org/605785 | 16:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | 16:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore forcing of evacuation for nested instance https://review.openstack.org/606111 | 16:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | 16:04 |
| efried | I guess that alone isn't worth holding it up since jay is +2 at this point. | 16:04 |
| efried | And I guess I do see that it becomes necessary again for nested. | 16:04 |
| efried | it's just a weird transition. | 16:04 |
| gibi | efried: totally agree that allocation_request_version is strange but I'm not 100% sure we simply drop it and fix the version in the report client globally | 16:06 |
| efried | gibi: IIUC, it's necessary whenever there's a possibility of the conductor and the scheduler being at different versions. | 16:06 |
| efried | ...versions of nova, leading to them being aware of different versions of placement. Placement ought to always be at least up to date with the later of the two. | 16:07 |
| *** mdbooth has joined #openstack-nova | 16:08 | |
| efried | I assume that's still a possible scenario, for like rolling upgrades or whatever. | 16:08 |
| gibi | efried: I'm not sure we support that scheduler and conductor on different version. What complicates it is that we send alternates around in RPC messages for reschedule | 16:08 |
| efried | I'm guessing there's a rule/guideline, like you have to upgrade the computes before the scheduler, or vice versa? dansmith do you know that answer? | 16:08 |
| gibi | efried: controller first is the ruel | 16:09 |
| gibi | rule | 16:09 |
| gibi | but I think controller services needs to be updated at once but I'm not sure | 16:09 |
| gibi | I have to run now. talk to you tomorrow | 16:09 |
| efried | thx gibi | 16:10 |
| gibi | efried: thank you | 16:10 |
| *** dtantsur is now known as dtantsur|afk | 16:10 | |
| openstackgerrit | Merged openstack/nova master: Skip test_parallel_evacuate_with_server_group until fixed https://review.openstack.org/607620 | 16:13 |
| openstackgerrit | Jack Ding proposed openstack/nova master: Handle missing marker during online data migration https://review.openstack.org/605164 | 16:13 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down https://review.openstack.org/584829 | 16:17 |
| *** slagle has joined #openstack-nova | 16:20 | |
| dansmith | efried: control services first and together, computes last and in any order you want | 16:23 |
| efried | dansmith: Okay, so the reason for allocation_request_version is in case computes are currently downlevel from controller and/or each other. Thanks. | 16:23 |
| openstackgerrit | Jack Ding proposed openstack/nova master: Add HPET timer support for x86 guests https://review.openstack.org/605902 | 16:24 |
| dansmith | efried: yeah | 16:24 |
| bauzas | efried: https://docs.openstack.org/nova/latest/user/upgrade.html | 16:24 |
| bauzas | HTH | 16:24 |
| bauzas | anyway, calling it a day | 16:25 |
| bauzas | do we have some hangout at 8pm UTC so ? if so, efried, could you please ping me on IRC ? | 16:25 |
| efried | bauzas: Yes, dansmith or I will ping you. | 16:25 |
| bauzas | cool cool | 16:26 |
| bauzas | see you later then. | 16:26 |
| efried | l8r | 16:26 |
| efried | or "bis später" :) | 16:26 |
| bauzas | heh, I won't finally have opportunity to speak German this year, but okay | 16:26 |
| *** Bhujay has joined #openstack-nova | 16:34 | |
| *** alexchadin has quit IRC | 16:34 | |
| *** dosaboy has quit IRC | 16:39 | |
| openstackgerrit | Gaudenz Steinlin proposed openstack/nova master: Ignore misleading resource updates from virt driver https://review.openstack.org/523006 | 16:41 |
| *** dosaboy has joined #openstack-nova | 16:45 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | 16:48 |
| openstackgerrit | melanie witt proposed openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | 16:52 |
| *** moshele has joined #openstack-nova | 16:53 | |
| *** sahid has quit IRC | 16:54 | |
| *** dosaboy has quit IRC | 16:55 | |
| *** Bhujay has quit IRC | 17:01 | |
| *** dosaboy has joined #openstack-nova | 17:01 | |
| *** derekh has quit IRC | 17:04 | |
| sean-k-mooney | dansmith: did ye have that hangout on hpet extraspec vs trait vs both ? | 17:04 |
| dansmith | sean-k-mooney: in two hours | 17:04 |
| sean-k-mooney | oh ok cool in that case i have time to read the specs properly | 17:05 |
| efried | dansmith: three hours? | 17:10 |
| dansmith | efried: 2000 UTC is 1200 PDT no? | 17:10 |
| dansmith | oh sorry, I was looking at dublin time | 17:11 |
| dansmith | yeah three hours | 17:11 |
| sean-k-mooney | ya dublin is currently on irish summer time | 17:12 |
| sean-k-mooney | hopefully daylight saving will no longer be a thing soon | 17:12 |
| openstackgerrit | Merged openstack/nova stable/ocata: nova-manage - fix online_data_migrations counts https://review.openstack.org/605842 | 17:14 |
| *** pcaruana has joined #openstack-nova | 17:17 | |
| *** mvkr has quit IRC | 17:20 | |
| *** jpena is now known as jpena|off | 17:21 | |
| *** sridharg has quit IRC | 17:22 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion bump for handling-down-cell https://review.openstack.org/591657 | 17:29 |
| *** Bhujay has joined #openstack-nova | 17:30 | |
| *** Swami has quit IRC | 17:36 | |
| openstackgerrit | Merged openstack/nova master: Add attach kwarg to base/nova-net allocate_for_instance methods https://review.openstack.org/605464 | 17:41 |
| openstackgerrit | melanie witt proposed openstack/nova master: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/608040 | 17:43 |
| *** tbachman has quit IRC | 17:44 | |
| *** tssurya has quit IRC | 17:48 | |
| *** tbachman has joined #openstack-nova | 17:57 | |
| *** pcaruana has quit IRC | 18:01 | |
| *** munimeha1_ has quit IRC | 18:01 | |
| *** tbachman has quit IRC | 18:03 | |
| *** Swami has joined #openstack-nova | 18:04 | |
| *** tbachman has joined #openstack-nova | 18:05 | |
| openstackgerrit | melanie witt proposed openstack/nova master: De-dupe subnet IDs when calling neutron /subnets API https://review.openstack.org/608040 | 18:06 |
| *** mvkr has joined #openstack-nova | 18:09 | |
| openstackgerrit | Merged openstack/nova master: consumer gen: support claim_resources https://review.openstack.org/583667 | 18:11 |
| *** mrch_ has joined #openstack-nova | 18:13 | |
| *** Bhujay has quit IRC | 18:20 | |
| *** imacdonn has quit IRC | 18:22 | |
| *** imacdonn has joined #openstack-nova | 18:22 | |
| openstackgerrit | Vlad Gusev proposed openstack/nova master: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/555160 | 18:25 |
| *** macza has quit IRC | 18:25 | |
| *** macza has joined #openstack-nova | 18:26 | |
| *** s10 has joined #openstack-nova | 18:34 | |
| *** mriedem has joined #openstack-nova | 18:34 | |
| *** tssurya has joined #openstack-nova | 18:36 | |
| *** moshele has quit IRC | 18:36 | |
| jaypipes | melwitt, dansmith, other stable cores... https://review.openstack.org/#/c/607626/ would be great to review. thanks in advance. | 18:38 |
| *** cdent has quit IRC | 18:39 | |
| s10 | Please approve back ports for https://review.openstack.org/#/q/I27160140cbccf3e6914aed2dcff54d547e446c1f and https://review.openstack.org/#/q/topic:bug/1785827+(status:open+OR+status:merged) | 18:40 |
| *** tbachman has quit IRC | 18:41 | |
| melwitt | jaypipes: +W | 18:46 |
| melwitt | mriedem: this backport just needs an upgrade of your +1 to a +2/+W https://review.openstack.org/592337 | 18:52 |
| *** ralonsoh has quit IRC | 18:52 | |
| jaypipes | melwitt: danke | 18:59 |
| *** tbachman has joined #openstack-nova | 18:59 | |
| jaypipes | efried: left a question/thought for you on https://review.openstack.org/#/c/605902/ | 18:59 |
| * efried clicks | 19:00 | |
| efried | jaypipes: For the sake of keeping things together, perhaps we should have that discussion on the spec. | 19:01 |
| jaypipes | efried: or the hangout later... | 19:02 |
| efried | jaypipes: I agree it makes sense to allow the trait to be in the image. Not sure that means we shouldn't also support it in the flavor, or that we should allow it to be present in the flavor and still hit a non-capable host. | 19:02 |
| jaypipes | efried: ack, totes | 19:02 |
| jaypipes | efried: and it's not like we have the ability to say "this trait can only go in an image metadata k/v and not a flavor extra spec k/v..." | 19:03 |
| efried | jaypipes: I assume we would support something like hw:hpet=True in image meta. | 19:03 |
| efried | so really any of the three options in the spec could be applied to both flavor and image | 19:03 |
| jaypipes | efried: I was just pointing out that for *some* of these things, the trait makes more sense as a constraint that is derived from an image, versus a constraint coming from a flavor... | 19:03 |
| efried | oh, yeah, totally agree with you there. | 19:03 |
| efried | If the op wants the flavor to be flexible in terms of which arch it lands on, don't put arch-specific stuff in it. | 19:04 |
| efried | that seems like a fair rule of thumb | 19:04 |
| jaypipes | ya | 19:04 |
| jaypipes | efried: I guess what I'm saying is I don't think a decision on the traits thing should hold up jding's patch. | 19:05 |
| efried | jaypipes: no, but the decision about whether to fail if I see the request but can't fulfil it. | 19:06 |
| efried | ^ should hold up the patch | 19:06 |
| jaypipes | efried: but again, if the guest/image arch isn't x86, the flavor's HPET extra spec makes no difference... | 19:06 |
| efried | so that's where I think we disagree. | 19:06 |
| efried | If you've asked for a thing in the flavor, you should get the thing, or fail. | 19:07 |
| efried | unless you want to call it hw:hpet_if_you_can_please | 19:07 |
| efried | jaypipes: I added a note here: https://review.openstack.org/#/c/607989/1/specs/stein/approved/support-hpet-on-guest.rst@23 | 19:08 |
| dansmith | are you guys cheating? | 19:09 |
| efried | ah dammit, I thought you said dansmith wasn't here | 19:09 |
| dansmith | I'm always here. | 19:09 |
| *** mriedem has quit IRC | 19:15 | |
| *** s10 has quit IRC | 19:17 | |
| jaypipes | efried: sorry, standup meetings... | 19:19 |
| efried | Well, you're a standup kind of guy. | 19:19 |
| jaypipes | lol | 19:23 |
| *** cdent has joined #openstack-nova | 19:23 | |
| *** mriedem has joined #openstack-nova | 19:24 | |
| *** s10 has joined #openstack-nova | 19:24 | |
| efried | btw, the irony is not lost on me that standup meetings, which are supposed to be standup so they're short, are consuming your whole day. | 19:25 |
| mriedem | fwiw, translating flavor extra specs that the driver needs to placement traits in a pre-request filter doesn't seem too hair brained, we've talked about the same kind of thing for translating volume multiattach boot requests to query allocation candidates via compute node capabilities | 19:25 |
| * mriedem read the meeting log | 19:26 | |
| dansmith | mriedem: yeah I said that in the meeting | 19:26 |
| dansmith | it's basically what request filters are for | 19:26 |
| mriedem | if we do it generically it should be ok - just don't add one filter per extra spec | 19:27 |
| mriedem | that would be hair brained | 19:27 |
| efried | mriedem: FYI we're doing a hangout in ~34min, and there's a spec https://review.openstack.org/#/c/607989/ where the options are laid out and we're collecting thoughts | 19:27 |
| dansmith | mriedem: you are not PTOing well | 19:27 |
| mriedem | i'm T-33 minutes until nephew gets home | 19:27 |
| mriedem | at which point i'll drop off | 19:27 |
| dansmith | not a good excuse | 19:27 |
| mriedem | after listening to AM radio for 4 hours in the car i needed something to do | 19:27 |
| efried | mriedem: Then you may register your vote in the spec, and we'll make up everything else you might have said. | 19:28 |
| dansmith | mriedem: do you have an opinion of not just adding another hw_ spec and making this a trait requirement in the flavor? | 19:28 |
| dansmith | so we get the scheduling for free | 19:28 |
| mriedem | but the driver still needs to know about the thing right? | 19:30 |
| *** munimeha1 has joined #openstack-nova | 19:30 | |
| dansmith | sure, it just looks for the presence of that on the flavor | 19:30 |
| mriedem | which we then figure out from the trait on the flavor rather than an extra spec? | 19:30 |
| dansmith | like we do for gpus today | 19:30 |
| dansmith | right | 19:30 |
| mriedem | is tha twhat we do for gpus today? | 19:30 |
| mriedem | i didn't know | 19:30 |
| dansmith | yeah | 19:30 |
| *** tbachman has quit IRC | 19:30 | |
| mriedem | then sure | 19:30 |
| efried | does jaypipes know about that ^ ? (the GPU thing) | 19:30 |
| mriedem | throw it in a method that knows what to look for and it doesn't matter if its extra spec or trait | 19:30 |
| sean-k-mooney | dansmith: today we enable a gpu by addign a trait? | 19:31 |
| jaypipes | efried: still standing... | 19:31 |
| dansmith | sean-k-mooney: no | 19:31 |
| dansmith | sean-k-mooney: it's a resource, but same difference | 19:31 |
| sean-k-mooney | dansmith: not quite | 19:31 |
| efried | I thought for GPUs we use conf to populate traits to get scheduling, and then the driver looks at the same conf ... | 19:31 |
| dansmith | we looked at the flavor initially, now I think we just inspect the allocation, but.. same sort of thing | 19:31 |
| efried | oh, yeah, if it's a resource vs traits that's totally different | 19:31 |
| sean-k-mooney | dansmith: a hpet is not a consumable resouce | 19:31 |
| efried | well, according to *some people* | 19:31 |
| dansmith | it seems entirely un-different to me | 19:31 |
| dansmith | sean-k-mooney: I'm aware, thanks :) | 19:32 |
| dansmith | the mechanism by which the driver knows to configure a thing is the same | 19:32 |
| efried | ftr I have no problem with that, been thinking along those lines since Denver-part-one, but Jay, Ed, and Chris all vetoed the idea pretty hard, then and since. | 19:32 |
| dansmith | I think the difference between your concern and theirs is whether or not the traits are opaque | 19:33 |
| dansmith | could be wrong, but that seems to be the stumbling block | 19:33 |
| efried | I'm not talking about key/value encoding at all. | 19:33 |
| dansmith | maybe we should wait for the hangout? | 19:34 |
| efried | I think the argument is that the trait is *not* opaque if the driver is doing a string compare to effect some config change on the guest.... sure :) | 19:34 |
| sean-k-mooney | dansmith: i was goint got comment but your right :) | 19:34 |
| dansmith | efried: I would respond to your comment with an argument, but I will wait | 19:34 |
| efried | Above was me being devil's advocate (for several specific values of "devil"). I don't feel that way. | 19:35 |
| efried | i.e. I think you and I are in agreement, dansmith | 19:35 |
| * dansmith shudders | 19:36 | |
| efried | ikr | 19:36 |
| efried | Swhy the whole ownership traits thing caused a rift in the continuum. | 19:36 |
| *** awaugama has quit IRC | 19:37 | |
| sean-k-mooney | efried: ya so did anything come of that? | 19:37 |
| efried | sean-k-mooney: I think it's dead. Ed and Jay are -1. | 19:38 |
| sean-k-mooney | i assumed it died on the vine at the ptg but i have not been following | 19:38 |
| efried | no, it was agreed upon at the ptg | 19:38 |
| efried | so I put up a patch for it, sec... | 19:38 |
| efried | sean-k-mooney: https://review.openstack.org/#/c/602160/ | 19:38 |
| sean-k-mooney | hehe i like the doc string | 19:39 |
| sean-k-mooney | traited is a ajative that is not used often enough | 19:40 |
| efried | Given Jay's (albeit reluctant) capitulation on the ML, we *might* still get away with using custom traits for ownership for a little while until we have that represented in some native key/value form (Jay's preference being an actual column in the resource provider table). | 19:40 |
| efried | but I don't see the standard traits flying. | 19:40 |
| jaypipes | ok, I'm back now.. sorry about that | 19:41 |
| * jaypipes reads back | 19:41 | |
| sean-k-mooney | was that not the original propsoal. add an owner colume to the resouce providers | 19:41 |
| dansmith | sean-k-mooney: it was, and it was NAKd | 19:41 |
| dansmith | it was for a different reason though | 19:41 |
| *** munimeha1_ has joined #openstack-nova | 19:42 | |
| dansmith | the consumer type/owner has to not be just "cinder", "nova" if we want to support the multi-service possibility, | 19:43 |
| dansmith | which means you really only know "is this mine or not" if you're looking at that field | 19:43 |
| dansmith | which is probably okay | 19:43 |
| *** slagle has quit IRC | 19:43 | |
| sean-k-mooney | oh but that would be an ower of an allocation not an ower of the resouce provider | 19:43 |
| *** munimeha1 has quit IRC | 19:43 | |
| dansmith | ah you're right I'm confusing these two things | 19:44 |
| efried | We do have to assume well-behaved API client code. And I don't think we want to, like, make the providers only appear to their owners, that would be crazy. | 19:45 |
| efried | so yeah, load up a provider record, look at the owner field, if it ain't me, don't dork with it. | 19:45 |
| sean-k-mooney | for owner of the resouce provder we disucssed 3 options. a trait. a nameing schem or a new db/object filed. the first two did not need placement change and the trait could be standardised so we suggeted that one | 19:45 |
| sean-k-mooney | at least i think that was the rough logic | 19:45 |
| efried | ++ | 19:46 |
| cdent | could (not saying it is a good idea, just being a completist) also use an aggregate | 19:46 |
| dansmith | yep | 19:46 |
| efried | But Jay was out of the room or something. | 19:46 |
| sean-k-mooney | cdent: yes that could also work | 19:46 |
| dansmith | efried: he was there, but doing the "sit in the back of the room looking like he wants to kill everyone" thing he does | 19:46 |
| sean-k-mooney | that came up in related to neutron and rack scale design before | 19:47 |
| *** eharney has quit IRC | 19:47 | |
| efried | oh, see, if he had been out of the room actually killing someone, that might have been a good excuse | 19:47 |
| dansmith | depends on who | 19:48 |
| efried | comes back in dripping blood, "Okay, now what decisions did y'all make without me?" | 19:48 |
| jaypipes | dansmith: s/kill someone/kill himself/ .... but yeah. that. | 19:48 |
| sean-k-mooney | jaypipes: arent you glad you rejoined chat | 19:49 |
| jaypipes | are we ready to hangout? | 19:49 |
| dansmith | https://hangouts.google.com/call/7dw4HGf7o0A9UahxcR3ZAAEE | 19:50 |
| dansmith | we can watch jaypipes make that look until it's time | 19:50 |
| melwitt | I'm gonna lurk | 19:50 |
| efried | cfriesen, sean-k-mooney, jackding: | 19:53 |
| cdent | dansmith, efried, jaypipes: i'm unable to join, but as I said before: the combined might of the brains involved ought to be able to get it right, but I do hope you will be summarizing to email or whatever please and thank you. | 19:53 |
| * cdent waves goodnight | 19:54 | |
| *** cdent has quit IRC | 19:54 | |
| sean-k-mooney | efried: sorry was making tea. joining | 19:54 |
| melwitt | bauzas: it's hangout time ^ | 19:55 |
| bauzas | melwitt: thanks | 19:58 |
| *** priteau has quit IRC | 20:15 | |
| *** s10 has quit IRC | 20:16 | |
| *** s10 has joined #openstack-nova | 20:16 | |
| *** s10 has quit IRC | 20:16 | |
| *** s10 has joined #openstack-nova | 20:17 | |
| *** s10 has quit IRC | 20:17 | |
| *** s10 has joined #openstack-nova | 20:18 | |
| *** s10 has quit IRC | 20:18 | |
| *** s10 has joined #openstack-nova | 20:18 | |
| *** s10 has quit IRC | 20:19 | |
| *** slagle has joined #openstack-nova | 20:19 | |
| *** s10 has joined #openstack-nova | 20:19 | |
| *** s10 has quit IRC | 20:20 | |
| *** med_ has joined #openstack-nova | 20:26 | |
| *** priteau has joined #openstack-nova | 20:36 | |
| *** devananda has quit IRC | 20:38 | |
| *** slagle has quit IRC | 20:39 | |
| *** macza has quit IRC | 20:40 | |
| *** macza has joined #openstack-nova | 20:41 | |
| *** med_ has quit IRC | 20:43 | |
| *** hamzy_ has quit IRC | 20:44 | |
| *** priteau has quit IRC | 20:48 | |
| *** mchlumsky has quit IRC | 20:58 | |
| *** erlon_ has quit IRC | 21:05 | |
| *** mriedem has quit IRC | 21:05 | |
| *** itlinux has quit IRC | 21:14 | |
| *** itlinux has joined #openstack-nova | 21:17 | |
| *** devananda has joined #openstack-nova | 21:25 | |
| openstackgerrit | iain MacDonnell proposed openstack/nova master: Handle online_data_migrations exceptions https://review.openstack.org/608091 | 21:30 |
| imacdonn | ^ dansmith: please review | 21:30 |
| openstackgerrit | Merged openstack/nova stable/ocata: Re-use existing ComputeNode on ironic rebalance https://review.openstack.org/607626 | 21:32 |
| *** spatel has quit IRC | 21:34 | |
| *** panda is now known as panda|off | 21:36 | |
| *** munimeha1_ has quit IRC | 21:45 | |
| cfriesen | dansmith: jaypipes: efried: sean-k-mooney: how does this look for a recap email? http://paste.openstack.org/raw/731538/ | 21:51 |
| cfriesen | and now I know what efried meant by getting spanked by Sigyn | 21:52 |
| efried | cfriesen: You could mention: | 21:54 |
| efried | - This approach only makes sense for simple booleans. Anything requiring more complex configuration will likely need additional extra_spec and/or config and/or unicorn dust. | 21:54 |
| efried | - If we do get down to libvirt, and the trait is set, and the driver for whatever reason determines it's not capable of flipping the switch, it should fail. | 21:54 |
| efried | and "HPET timer" is redundant :P | 21:55 |
| efried | otherwise, lgtm. | 21:55 |
| cfriesen | efried: cool, will add that. | 21:55 |
| efried | cfriesen: And you/jack are going to take over the spec, yah? | 21:56 |
| cfriesen | efried: yep | 21:57 |
| efried | coo | 21:58 |
| efried | imacdonn: Where does the log go when you run that? | 21:59 |
| imacdonn | efried: nova-manage.log | 21:59 |
| imacdonn | efried: or, I guess, wherever logging puts the log named "nova-manage" | 22:00 |
| efried | hmph. Where is that set up? | 22:00 |
| imacdonn | efried: schema migrations stuff already gets logged there | 22:00 |
| efried | I mean, I assume you tried this out locally and were able to see those logs | 22:00 |
| efried | okay | 22:00 |
| imacdonn | efried: affirm | 22:00 |
| jaypipes | cfriesen: yes, nice work on that ML post. ++ | 22:01 |
| efried | imacdonn: oic, it gets set up in main() | 22:01 |
| imacdonn | efried: ah, yeah | 22:02 |
| openstackgerrit | Merged openstack/nova master: Don't emit warning when ironic properties are zero https://review.openstack.org/605754 | 22:05 |
| openstackgerrit | Merged openstack/nova stable/pike: Fix service list for disabled compute using MC driver https://review.openstack.org/592337 | 22:05 |
| *** itlinux has quit IRC | 22:05 | |
| efried | cfriesen: When you're spiffing up that spec, I forgot to put blueprint: support-hpet-on-guest in the commit message | 22:06 |
| efried | interestingly, lp seems to have picked it up anyway. Maybe based on the topic? | 22:07 |
| cfriesen | I noticed that too. gotta be the topic | 22:07 |
| cfriesen | but yes, will add it in | 22:07 |
| efried | okay, folks, I'm done like toast. ō/ | 22:08 |
| imacdonn | you're fried? fnar-fnar | 22:08 |
| cfriesen | jaypipes: so would we need to propose changes in os-traits adding new traits for HPET and TPM? | 22:10 |
| *** med_ has joined #openstack-nova | 22:11 | |
| jaypipes | cfriesen: go for it :) | 22:13 |
| cfriesen | jaypipes: just wondering what the "proper" way to do this is | 22:13 |
| *** tssurya has quit IRC | 22:14 | |
| cfriesen | is anything in nova using os-traits yet? | 22:16 |
| *** itlinux has joined #openstack-nova | 22:22 | |
| *** itlinux has quit IRC | 22:25 | |
| cfriesen | what do you guys think of using HW_SYSTEM_X86_* and HW_SYSTEM_* for instance traits that aren't cpu/gpu/nic specific, like HPET and TPM? | 22:26 |
| cfriesen | and uefi | 22:27 |
| *** rcernin has joined #openstack-nova | 22:27 | |
| *** plestang has quit IRC | 22:28 | |
| cfriesen | if we have a trait that is requested in the flavor and forbidden in the image, do we error out currently? | 22:41 |
| *** tbachman has joined #openstack-nova | 22:57 | |
| *** macza has quit IRC | 22:58 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | 23:21 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | 23:22 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 23:23 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | 23:23 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106 | 23:23 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110 | 23:23 |
| *** mlavalle has quit IRC | 23:32 | |
| openstackgerrit | Merged openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672 | 23:33 |
| *** slagle has joined #openstack-nova | 23:35 | |
| *** edmondsw has quit IRC | 23:38 | |
| openstackgerrit | Merged openstack/nova master: Use provider tree in virt FakeDriver https://review.openstack.org/604083 | 23:38 |
| *** edleafe has quit IRC | 23:39 | |
| *** takashin has joined #openstack-nova | 23:48 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for examples https://review.openstack.org/604060 | 23:52 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628 | 23:52 |
| *** tetsuro_ has joined #openstack-nova | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!