| *** takashin has joined #openstack-placement | 00:06 | |
| *** takashin has quit IRC | 00:07 | |
| *** takashin has joined #openstack-placement | 00:08 | |
| *** takashin has left #openstack-placement | 01:32 | |
| *** tetsuro has joined #openstack-placement | 01:43 | |
| openstackgerrit | Merged openstack/nova master: Fix typo in enable_certificate_validation config option help https://review.openstack.org/572185 | 01:55 |
|---|---|---|
| openstackgerrit | Merged openstack/nova master: Mention running rootwrap in daemon mode if hitting vif plug timeouts https://review.openstack.org/571577 | 02:04 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240 | 02:29 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398 | 03:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: libvirt live migration: fit NUMA topology on dest https://review.openstack.org/567242 | 03:09 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723 | 03:09 |
| openstackgerrit | chenxiangui proposed openstack/nova master: Replace a extra_specs with an extra_specs https://review.openstack.org/574535 | 03:34 |
| openstackgerrit | qingszhao proposed openstack/os-traits master: change default python 3 env in tox to 3.5 https://review.openstack.org/574539 | 03:42 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: DNM: default [xenserver]/image_handler to vdi_remote_stream https://review.openstack.org/574318 | 04:59 |
| openstackgerrit | garyk proposed openstack/nova master: VMware: save VC reads for information that is static https://review.openstack.org/574552 | 05:34 |
| openstackgerrit | garyk proposed openstack/nova master: VMware: remove reading resourcePool data https://review.openstack.org/574553 | 05:37 |
| openstackgerrit | jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243 | 06:33 |
| *** ttsiouts has joined #openstack-placement | 07:11 | |
| *** e0ne has joined #openstack-placement | 07:12 | |
| openstackgerrit | Pooja Jadhav proposed openstack/nova master: Remove unnecessary parameters from create volume API https://review.openstack.org/574598 | 07:14 |
| *** e0ne has quit IRC | 07:15 | |
| openstackgerrit | Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317 | 07:26 |
| *** e0ne has joined #openstack-placement | 07:26 | |
| *** e0ne has quit IRC | 07:29 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403 | 08:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404 | 08:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405 | 08:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406 | 08:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407 | 08:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_microversion https://review.openstack.org/574408 | 08:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Do not use nova.test in placement.test_deploy https://review.openstack.org/574409 | 08:02 |
| *** e0ne has joined #openstack-placement | 08:08 | |
| *** e0ne has quit IRC | 08:08 | |
| openstackgerrit | XiaojueGuan proposed openstack/nova master: Update links in README https://review.openstack.org/574644 | 08:35 |
| *** ttsiouts has quit IRC | 08:37 | |
| *** ttsiouts has joined #openstack-placement | 08:37 | |
| *** cdent has joined #openstack-placement | 08:42 | |
| *** ttsiouts has quit IRC | 08:44 | |
| *** ttsiouts has joined #openstack-placement | 08:46 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092 | 08:52 |
| *** ttsiouts has quit IRC | 09:02 | |
| *** ttsiouts has joined #openstack-placement | 09:02 | |
| *** ttsiouts has quit IRC | 09:12 | |
| *** nicolasbock has joined #openstack-placement | 09:18 | |
| *** ttsiouts has joined #openstack-placement | 09:21 | |
| *** ttsiouts has quit IRC | 09:24 | |
| *** ttsiouts has joined #openstack-placement | 09:28 | |
| *** e0ne has joined #openstack-placement | 09:31 | |
| *** purplerbot has quit IRC | 09:31 | |
| *** purplerbot has joined #openstack-placement | 09:31 | |
| *** nicolasbock has quit IRC | 09:37 | |
| bhagyashris_ | gibi: Hi, Actaully i am trying to reproduce the issue as you mentioned on patch https://review.openstack.org/#/c/560459/ . What i am trying to do is put the pdb https://github.com/openstack/nova/blob/3693caa54132ef3ec278d846dc50943a8d52509b/nova/virt/libvirt/imagebackend.py#L371 and ran my test but not able to reach there and passed the test successfully | 09:54 |
| *** purplerbot has quit IRC | 09:56 | |
| *** purplerbot has joined #openstack-placement | 09:56 | |
| *** ttsiouts has quit IRC | 10:03 | |
| bhagyashris_ | gibi: and looking at the error for the test on zuul for patch https://review.openstack.org/#/c/560459/ seems that the host uuid is not found so i am little bit confuse about how it's not getting the instance_path | 10:04 |
| *** ttsiouts has joined #openstack-placement | 10:12 | |
| *** ttsiouts has quit IRC | 10:14 | |
| *** ttsiouts has joined #openstack-placement | 10:16 | |
| openstackgerrit | garyk proposed openstack/nova master: VMware: populate datastore refs at init https://review.openstack.org/574688 | 10:24 |
| gibi | bhagyashris_: you can reproduce the problem by deleting the 'instances' dir in your nova repo and you run the only functional test you added in your patch | 10:38 |
| gibi | bhagyashris_: OR make the instances dir unreadable and unwriteable (e.g. chmod -rxw ./instances) then run the full functional test suite | 10:39 |
| gibi | bhagyashris_: you test does not reach the imagebackend code other tests are hitting that. This is exaclty why your test only passes if the other tests hitting the image backend runs before your test | 10:40 |
| *** ttsiouts has quit IRC | 10:49 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move db using provider unit tests to functional https://review.openstack.org/572280 | 10:57 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Adjust db using allocation unit tests https://review.openstack.org/572281 | 10:57 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Remove nova dependencies from test_resource_provider https://review.openstack.org/572282 | 10:57 |
| gibi | bhagyashris_: and the first error is the instances_path in the log http://logs.openstack.org/59/560459/16/gate/nova-tox-functional/c04cbeb/job-output.txt.gz#_2018-06-08_20_26_17_860504 | 10:59 |
| cdent | efried: in case you're nearby: I'm taking the lock on squashing the placement direct stack unless you're already in progress | 11:00 |
| openstackgerrit | jiang wei proposed openstack/nova master: Add action initiator attribute to the payload https://review.openstack.org/536243 | 11:03 |
| openstackgerrit | Merged openstack/nova stable/queens: Change consecutive build failure limit to a weigher https://review.openstack.org/573239 | 11:21 |
| *** nicolasbock has joined #openstack-placement | 11:33 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Provide a direct interface to placement https://review.openstack.org/572576 | 11:36 |
| bhagyashris_ | gibi: ok i will check immediately and will let you know | 11:37 |
| gibi | bhagyashris_: OK, cool | 11:38 |
| cdent | gibi: your thoughts on placement direct ^^ would be great | 11:40 |
| gibi | cdent: looking... | 11:40 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487 | 11:46 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series https://review.openstack.org/573604 | 11:46 |
| *** tetsuro has quit IRC | 11:47 | |
| bhagyashris_ | gibi: Hi, i have deleted the ./instances/ repo and ran my test but it's passing | 11:49 |
| gibi | bhagyashris_: did the ./instances directory now exists after your test run? | 11:50 |
| bhagyashris_ | gibi: no it's not exists after runinng my test | 11:53 |
| gibi | bhagyashris_: then I don't get it. Let me reproduce it again locally and save all the steps in a paste | 11:54 |
| bhagyashris_ | gibi: ok | 11:54 |
| *** ttsiouts has joined #openstack-placement | 11:56 | |
| gibi | cdent: will we ever use two placement direct in the same process? if yes did we considered the global state like resource_provider._TRAITS_SYNCED as a source of possible problems? | 11:59 |
| *** ttsiouts has quit IRC | 12:01 | |
| cdent | gibi: good question. as with tests, I would think the caller would need to be responsible for those decisions. In "real" use in the same process, nothing would need to change: if traits are unsynced, they'll be synced the first time and that's it | 12:03 |
| *** edmondsw has joined #openstack-placement | 12:03 | |
| gibi | cdent: I see, there is a lock in _ensure_trait_sync that will save us | 12:05 |
| cdent | gibi: yeah, and eventually that will change a bit: https://review.openstack.org/#/c/553857/ | 12:05 |
| bhagyashris_ | gibi: i am able to reproduce the problem | 12:07 |
| leakypipes | efried: I'm +2 on https://review.openstack.org/#/c/572282/ (cdent's unit/func test refactor series) | 12:14 |
| leakypipes | gibi: ^^ if you want to have a final look | 12:14 |
| cdent | leakypipes: you decided it should be friday every day? (good choice) | 12:15 |
| gibi | leakypipes: putting it in my review queue | 12:15 |
| *** leakypipes is now known as jaypipes | 12:15 | |
| * cdent has a sad | 12:15 | |
| jaypipes | cdent: heh | 12:15 |
| cdent | jaypipes: I'm very curious about your new article proposal, do you have theories going into it, or are you trying to avoid biasing yourself? | 12:15 |
| jaypipes | cdent: this one? https://github.com/jaypipes/articles/issues/37 or this one? https://github.com/jaypipes/articles/issues/38 | 12:16 |
| cdent | 38 | 12:17 |
| cdent | I'm curious about 37 too, but 38 is the one that just crossed my radar | 12:17 |
| *** ttsiouts has joined #openstack-placement | 12:17 | |
| jaypipes | cdent: the answer to both is that I am intentionally trying to avoid biases and honestly find some good solid data for the particular users I am concerned about (IT admins and traditional company sysops) | 12:18 |
| jaypipes | cdent: there is a dearth of good data and information out there comparing costs (in various definitions of "cost") for cloud and non-cloud infrastructure. | 12:18 |
| jaypipes | cdent: for very commonly-deployed applications and not the "cloud-native from the start" type applications. | 12:18 |
| cdent | true. plenty of pontificating though. | 12:18 |
| gibi | cdent: OK after reading a bit around trait sync and rc cache I think we are safe. I'm +2 on placement direct | 12:19 |
| jaypipes | cdent: right. I'd like to not pontificate -- as much as possible -- and just let the experience and data inform the article. that's the objective at least. | 12:19 |
| *** ttsiouts has quit IRC | 12:19 | |
| gibi | bhagyashris_: what was the trick? | 12:19 |
| jaypipes | cdent: same with the hierarchical data article. | 12:19 |
| jaypipes | cdent: and since I don't officially start at my new job until the 25th, I'm going to carve out time over the next couple weeks to work on both. | 12:20 |
| cdent | gibi: cool, thanks | 12:20 |
| bhagyashris_ | gibi: that was my mistake I was running different test | 12:20 |
| cdent | jaypipes: do you have a new new job? | 12:20 |
| gibi | bhagyashris_: OK, cool | 12:20 |
| jaypipes | cdent: yeah. | 12:21 |
| jaypipes | cdent: my last day at Verizon was Friday. | 12:21 |
| cdent | congratulations? | 12:21 |
| bhagyashris_ | gibi: Actually i am trying to fix this issue but not getting where exactly the fix will need can you guide me | 12:21 |
| jaypipes | cdent: I'm taking a couple weeks off to get my head back straight and work on some personal projects (like those articles) | 12:21 |
| *** ttsiouts has joined #openstack-placement | 12:21 | |
| jaypipes | cdent: I'll reserve the congratulations answer until I find out :) | 12:22 |
| cdent | +3 | 12:22 |
| jaypipes | flaming sword +5 | 12:22 |
| gibi | jaypipes: good luck with your new job! | 12:22 |
| jaypipes | gibi: ty. | 12:22 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 12:25 |
| cdent | gibi: fixed the moved test in ^ | 12:25 |
| cdent | jaypipes: efried and I ended up working together on the latest version of https://review.openstack.org/#/c/572576/ so that's taken him a bit out of the review loop, so if you've got some brain to share on placement direct, it's close, assuming the model is what we like | 12:27 |
| gibi | cdent: thanks! | 12:27 |
| gibi | bhagyashris_: as your new test configures using libvirt driver (that needs the path) here https://review.openstack.org/#/c/560459/16/nova/tests/functional/libvirt/test_shared_resource_provider.py@30 I would assume that creating the pre-conditions for such driver would go to that setUp as wll | 12:29 |
| gibi | bhagyashris_: s/pre-conditions/pre-requisitis/ | 12:30 |
| jaypipes | cdent: rock on. will definitely review. | 12:33 |
| bhagyashris_ | gibi: I have fix the test what i have did is just added self.useFixture( | 12:41 |
| bhagyashris_ | fixtures.MockPatch('nova.virt.libvirt.utils.get_fs_info')) this line in steUp() method of my test and ran the test so it's passing :) | 12:41 |
| bhagyashris_ | gibi: and if i remove the line as mentioned above ^ then it's failing with the same error as on zuul | 12:42 |
| cdent | jaypipes: you know much about ONAP HAS? https://wiki.onap.org/pages/viewpage.action?pageId=16005528 | 12:43 |
| gibi | bhagyashris_: that seems OK to me | 12:43 |
| gibi | bhagyashris_: push it and I can take a look | 12:44 |
| bhagyashris_ | gibi: yes I will push the patch soon | 12:44 |
| jaypipes | cdent: you don't want to get me started on ONAP. | 12:51 |
| cdent | ikr | 12:51 |
| jaypipes | cdent: I'm trying to stay in my happy place for 2 weeks. | 12:51 |
| cdent | HAS is on a list of random things that I'm vaguely supposed to be aware of if I happen to have the time, and thus rather low on the list. | 12:52 |
| *** efried has quit IRC | 12:52 | |
| *** efried has joined #openstack-placement | 12:53 | |
| efried | cdent: Giving a look at the squashed placement direct now. | 12:53 |
| gibi | cdent: nice patches in the series https://review.openstack.org/#/c/572282 I sent them to the gate | 12:56 |
| *** ttsiouts has quit IRC | 12:56 | |
| cdent | hurrah, thanks gibi and efried | 12:56 |
| jaypipes | cdent: ONAP is an over-architected piece of AT&T-internal home-grown telco love-fest. | 12:58 |
| cdent | jaypipes: yeah, that part I know well, but HAS I haven't looked at much until very recently | 12:58 |
| jaypipes | cdent: I replaced my original wording with "home-grown telco love-fest" there. | 12:58 |
| cdent | noted | 12:58 |
| *** ttsiouts has joined #openstack-placement | 12:59 | |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 13:00 |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 13:04 |
| bhagyashris_ | gibi, jaypipes: Pushed the patch https://review.openstack.org/560459 | 13:04 |
| jaypipes | bhagyashris_: +2 | 13:10 |
| bhagyashris_ | jaypipes: thank you :) | 13:10 |
| *** ttsiouts has quit IRC | 13:22 | |
| cdent | gibi: if you liked that one test refactor stack, you might like this one too: https://review.openstack.org/#/c/574403/ | 13:25 |
| efried | cdent, gibi: I was still thinking to fast-approve those, but it'll be trivial for gibi to look if you're so inclined. | 13:28 |
| cdent | efried: it was more for gibi's enjoyment, since he seemed to enjoy the others. | 13:28 |
| efried | :) | 13:28 |
| *** ttsiouts has joined #openstack-placement | 13:30 | |
| gibi | cdent, efried: thanks for taking care of my enjoyment :) | 13:30 |
| gibi | efried: I've put https://review.openstack.org/#/c/574403/ in my queue but it is behind of at least two other things so I'm fine if you go ahead and fast approve it | 13:31 |
| efried | okay. | 13:31 |
| efried | gibi: I fast-approved the bottom six, but if you could please keep https://review.openstack.org/#/c/574409/ in your queue. | 13:32 |
| efried | gibi: That one will also take you all of ten seconds, but it's marginally more involved so I didn't want to push it myself. | 13:33 |
| bhagyashris_ | efried: Hi, fixed the zuul failure on https://review.openstack.org/560459 :) | 13:33 |
| efried | bhagyashris_: I saw, thank you. | 13:33 |
| gibi | bhagyashris_: I will go and look at your fix still today | 13:33 |
| bhagyashris_ | gibi: ok thank you :) | 13:33 |
| gibi | efried: OK, will will take a look at https://review.openstack.org/#/c/574409/ | 13:34 |
| *** mriedem has joined #openstack-placement | 13:36 | |
| gibi | efried: it was about 30 seconds to look through what self.flags does but +2 +W ;) | 13:36 |
| efried | gibi: thanks | 13:36 |
| * gibi feels that today is a productive day | 13:36 | |
| cdent | gibi, efried: self.flags is one of those test helpers that I'm not sure helps, exactly because you have to go look at what it does | 13:39 |
| cdent | but then I'm one of those crazy "dry isn't always right in tests" crazies | 13:39 |
| gibi | cdent: I share your feeling, it is just a different api for set_override | 13:39 |
| *** ttsiouts has quit IRC | 13:39 | |
| efried | cdent: I'm with you on that. I actually re-looked at it yesterday for the first time in a while and realized just how thin it is. | 13:40 |
| *** ttsiouts has joined #openstack-placement | 13:40 | |
| cdent | it is handy if you want to set multiples | 13:40 |
| gibi | cdent: but in the other hand its name hides the fact that you are defining configuration for your test | 13:41 |
| cdent | yes | 13:41 |
| cdent | the name is a large part of the problem | 13:41 |
| cdent | of course if we really want to talk about problems: global CONF-- !!! | 13:41 |
| cdent | :) | 13:41 |
| cdent | gibi: is this what a productive day looks like: https://review.openstack.org/#/q/status:open+owner:%22Chris+Dent+%253Ccdent%2540anticdent.org%253E%22 | 13:42 |
| gibi | cdent: I envy your day now :) | 13:43 |
| cdent | that's all thanks to you and efried | 13:44 |
| cdent | and jaypipes too | 13:44 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove network info stubbing in functional test https://review.openstack.org/570018 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 13:55 |
| bhagyashris_ | jaypipes: Hi, I want to discuss regarding Create HW_CPU_HYPERTHREADING trait as mentioned in the spec https://review.openstack.org/#/c/555081/15 , so I have checked how to add the traits in os-trait lib so all hw cpu related traits are added under -> https://github.com/openstack/os-traits/tree/master/os_traits/hw/cpu but under this the cpu specific module are created ane added | 13:55 |
| bhagyashris_ | jaypipes: so where I should add HW_CPU_HYPERTHREADING ? | 13:56 |
| bhagyashris_ | jaypipes: As the traits name are according to the cpu type for eg. traits in https://github.com/openstack/os-traits/blob/master/os_traits/hw/cpu/aarch64.py are name as 'HW_CPU_AARCH64_ASIMDDP' | 13:59 |
| gibi | bhagyashris_: thanks of the fix, it works for me too. I'm +W on https://review.openstack.org/#/c/560459 | 14:01 |
| efried | woot | 14:01 |
| openstackgerrit | Simon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530 | 14:02 |
| cdent | bhagyashris_: yay! that's a _huge_ leap forward. It's something we've been talking about for years and years | 14:02 |
| bhagyashris_ | gibi: thank you :) | 14:04 |
| gibi | bhagyashris_: thank you! :) | 14:05 |
| bhagyashris_ | cdent: ohh | 14:06 |
| bhagyashris_ | cdent: I am not aware about past talk just started the work so eager to get inputs | 14:08 |
| efried | cdent: Yeah, we should brag on that some. In the placement update would be a good spot for it. (It isn't a placement change itself, but all of the placement work we've done to date is what made it possible.) | 14:08 |
| cdent | efried: yah, some bragging definitely in order | 14:08 |
| jaypipes | bhagyashris_: the HW_CPU_HYPERTHREADING trait would go in the dunderinit file /os_traits/hw/cpu/__init__.py | 14:09 |
| bhagyashris_ | jaypipes; ok. same thought was in my mind :) | 14:10 |
| bhagyashris_ | jaypipes: thank you :) | 14:11 |
| gibi | cdent, efried: maybe invoke success bot to add an entry here https://wiki.openstack.org/wiki/Successes :) | 14:11 |
| efried | that's a good idea gibi. | 14:11 |
| gibi | jaypipes, efried, cdent: now I think I understood the logic in https://review.openstack.org/#/c/573113 | 14:17 |
| gibi | efried: do you want do the honours and put a +W? | 14:17 |
| efried | gibi: No, you go for it. | 14:18 |
| gibi | efried: done | 14:19 |
| efried | thank you sir | 14:19 |
| openstackgerrit | Merged openstack/nova master: Move db using provider unit tests to functional https://review.openstack.org/572280 | 14:23 |
| openstackgerrit | Merged openstack/nova master: Adjust db using allocation unit tests https://review.openstack.org/572281 | 14:23 |
| openstackgerrit | Merged openstack/nova master: Remove nova dependencies from test_resource_provider https://review.openstack.org/572282 | 14:23 |
| *** ttsiouts has quit IRC | 14:25 | |
| *** ttsiouts has joined #openstack-placement | 14:32 | |
| *** ttsiouts has quit IRC | 14:35 | |
| *** ttsiouts has joined #openstack-placement | 14:40 | |
| openstackgerrit | Merged openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 14:49 |
| openstackgerrit | Merged openstack/nova master: Do not use nova.test in placement.test_util https://review.openstack.org/574403 | 14:54 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test heartbeat code in pre_live_migration https://review.openstack.org/574482 | 14:55 |
| efried | cdent: I looked throught PlacementDirect again and have some improvements. Want me to edit in place or throw 'em on top? | 15:04 |
| cdent | I'd say in place, unless there's some powerful reason not to? | 15:04 |
| efried | cdent: k, doin it. | 15:05 |
| efried | cdent: thought you might want to see the comments first :) | 15:05 |
| cdent | oh, you left that part out | 15:06 |
| cdent | looks like gerrit emails are slow efried, yeah, lemme read | 15:06 |
| openstackgerrit | Merged openstack/nova master: Do not use nova.test in placement.handlers.test_aggregate https://review.openstack.org/574404 | 15:09 |
| openstackgerrit | Merged openstack/nova master: Do not use nova.test in placement.test_requestlog https://review.openstack.org/574405 | 15:09 |
| openstackgerrit | Merged openstack/nova master: Do not use nova.test in placement.test_fault_wrap https://review.openstack.org/574406 | 15:09 |
| openstackgerrit | Merged openstack/nova master: Do not use nova.test in placement.test_handler https://review.openstack.org/574407 | 15:10 |
| openstackgerrit | Alexandre arents proposed openstack/nova master: Preserve images_type of instance during live migration https://review.openstack.org/570528 | 15:13 |
| cdent | efried: yeah, some other ideas in my responses | 15:15 |
| efried | cdent: I don't understand the suggestion after "blargh" | 15:17 |
| efried | cdent: Are you suggesting wrapping Adapter.request to always override raise_exc=False? | 15:17 |
| cdent | can raise_exc=False be set globally on the adapter so that it doesn't have to be per request? | 15:17 |
| cdent | I'm _hoping_ there is a way to set it is a default without needing to wrap a bunch of stuff | 15:18 |
| efried | I just checked. Sorry, no. | 15:18 |
| cdent | that's _horrible_ | 15:18 |
| cdent | is this a ksa thing? | 15:18 |
| efried | cdent: Yeah, looks like it. | 15:19 |
| efried | cdent: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L866-L869 | 15:20 |
| cdent | yeah, looking at the code ow | 15:20 |
| efried | cdent: I'm sure we could propose a change to ksa to allow that to be set globally on the Session and/or Adapter. | 15:20 |
| efried | because yeah, it kinda sucks. | 15:20 |
| efried | cdent: We would only have to subclass Adapter.request to override it. | 15:21 |
| cdent | true | 15:21 |
| cdent | and not much of an override | 15:21 |
| efried | nope | 15:21 |
| efried | But | 15:21 |
| efried | it's probably not The Right Thing | 15:22 |
| cdent | howabout: do that, and leave a todo about moving the concept to ksa | 15:22 |
| cdent | but. it's. so. wrong. | 15:22 |
| efried | if what we want to advertise is that we're producing an Adapter, we should produce an Adapter. | 15:22 |
| efried | Yeah, so we should fix it at ksa. | 15:22 |
| cdent | okay, then, in that case, definitely leave a big fat todo or fixme in the tests that are having to set it | 15:23 |
| efried | ight | 15:23 |
| cdent | I haven't got the cycles to change ksa anything like today or even this week. do you? | 15:23 |
| cdent | I'll do my part by making a bug, though? | 15:24 |
| efried | cdent: Sure | 15:25 |
| cdent | on it | 15:25 |
| efried | you seem to feel strongly about this. Can't wait to see what you say about it :) | 15:31 |
| cdent | I was chilled | 15:32 |
| cdent | https://bugs.launchpad.net/keystoneauth/+bug/1776501 | 15:32 |
| openstack | Launchpad bug 1776501 in keystoneauth "session.request has bad default for raise_exc when creating Adapter-based clients" [Undecided,New] | 15:32 |
| efried | cdent: Would you rather set it on the Adapter or the Session (or both?) | 15:37 |
| cdent | I'm agnostic. I just want to be able set it once, and forget about it | 15:37 |
| efried | k, we'll do it at the Adapter and see if it flies. | 15:38 |
| cdent | in the bug I talked about in term of the session because that's where it is, but *shrug* | 15:39 |
| cdent | man, the rules for when you are supposed to commit in the middle of a rebase and when not... | 15:40 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 15:41 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 15:41 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 15:41 |
| efried | cdent: Do you use git restack? | 15:41 |
| cdent | not usually, no | 15:41 |
| efried | cdent: It takes care of that for you; if you always say rebase --continue, it'll commit if you need a commit. | 15:42 |
| cdent | I feel bad enough about using git review (because it means there's magical invisible git stuff happening), I'm not sure I could cope with that | 15:42 |
| cdent | I want to know the rules | 15:42 |
| cdent | in any case, this was during the rebase that git review does, not my own rebase | 15:43 |
| efried | o | 15:43 |
| efried | I switched off that auto-rebase a while ago, for that reason. | 15:43 |
| cdent | I wanted it in this case because I know there was stuff pending. But then promptly forgot the rules. | 15:44 |
| efried | interesting, I always want to make sure I'm doing those rebases manually. | 15:44 |
| * cdent rebases again | 15:44 | |
| cdent | yeah, it's rare I want to do it that way | 15:45 |
| efried | Would seem that we're criss-cross from where one would expect on this :) | 15:45 |
| *** ttsiouts has quit IRC | 15:45 | |
| cdent | which stereotypes are we failing to represent this time? | 15:45 |
| *** ttsiouts has joined #openstack-placement | 15:46 | |
| *** ttsiouts has quit IRC | 15:48 | |
| *** ttsiouts has joined #openstack-placement | 15:48 | |
| edleafe | When I have a series, I write a little shell script to handle the steps. This way I save the typing (and typos) without having to rely on magic | 15:50 |
| *** ttsiouts has quit IRC | 15:52 | |
| efried | cdent: https://review.openstack.org/574784 raise_exc default in Adapter | 15:52 |
| cdent | I rarely have any issues with results, just confusion when I forget the rules. | 15:52 |
| cdent | that was speedy efried | 15:54 |
| *** ttsiouts has joined #openstack-placement | 15:54 | |
| efried | cdent: Not as speedy as mordred's response on your bug report. | 15:54 |
| cdent | i pinged him about it | 15:55 |
| cdent | efried: evoked you in a question to dansmith on the reshape spec | 15:56 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 15:56 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 15:56 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 15:56 |
| efried | cdent: Where? | 15:57 |
| cdent | line 89: https://review.openstack.org/#/c/572583/7/specs/rocky/approved/reshape-provider-tree.rst | 15:58 |
| *** ttsiouts has quit IRC | 15:59 | |
| *** e0ne has quit IRC | 15:59 | |
| cdent | biab | 15:59 |
| openstackgerrit | Merged openstack/nova master: Extract part of PlacementFixture to placement https://review.openstack.org/568359 | 16:00 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Provide a direct interface to placement https://review.openstack.org/572576 | 16:05 |
| efried | cdent, gibi: Those little tweaks ^ | 16:05 |
| openstackgerrit | Merged openstack/nova master: placement: Use INNER JOIN for requied traits https://review.openstack.org/573113 | 16:05 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add method to get cpu traits https://review.openstack.org/560317 | 16:08 |
| *** smcginnis[m] has quit IRC | 16:08 | |
| *** purplerbot has quit IRC | 16:11 | |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 16:11 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 16:11 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 16:11 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 16:11 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 16:11 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072 | 16:23 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284 | 16:23 |
| openstack | bug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,In progress] https://launchpad.net/bugs/1764883 - Assigned to Lee Yarwood (lyarwood) | 16:23 |
| gibi | efried: I'll look at it tomorrow | 16:30 |
| efried | gibi: sure, thanks | 16:30 |
| *** smcginnis[m] has joined #openstack-placement | 16:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: expand on various bdm parameters https://review.openstack.org/574805 | 16:51 |
| openstackgerrit | Simon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530 | 16:53 |
| *** purplerbot has joined #openstack-placement | 16:56 | |
| openstackgerrit | Merged openstack/nova master: Use instance project/user when creating RequestSpec during resize reschedule https://review.openstack.org/571245 | 17:07 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 17:12 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 17:12 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 17:12 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 17:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | 17:15 |
| openstackgerrit | Merged openstack/nova master: libvirt: Don't report DISK_GB if sharing https://review.openstack.org/560459 | 17:24 |
| jaypipes | I really need to find less gas-producing pugs. | 17:29 |
| *** markguz has joined #openstack-placement | 17:30 | |
| cdent | feed them beano? | 17:30 |
| markguz | hi there. i'm a bit behind on the releases and am just starting the upgrade process from newton to ocata. | 17:31 |
| markguz | I thought i'd configure the placement API first before going to ocata, and have kinda broken my nova now. | 17:31 |
| markguz | i followed some online threads and the ocata instructions as I couldn;t find any good instructions for Newton | 17:32 |
| jaypipes | cdent: :) | 17:32 |
| markguz | can anyone point me to the install guide for placement-api in newton? | 17:32 |
| jaypipes | markguz: whatcha running into? can you pastebin any errors you are running into? | 17:33 |
| markguz | was one ever written? | 17:33 |
| jaypipes | markguz: what are you using for deploying/installing openstack? | 17:33 |
| markguz | well i get stuck at the cell creation bit because there's no create_cell command | 17:34 |
| markguz | well once upon a time i used rdo to install kilo, and now that install has become newton, but i've just been doing rolling upgrades with yum | 17:34 |
| jaypipes | markguz: what instructions are you currently following? and what distribution of OpenStack are you installing/upgrading? | 17:34 |
| jaypipes | ok, RDO | 17:35 |
| *** e0ne has joined #openstack-placement | 17:35 | |
| markguz | jaypipes: i have newton, so i figured before going to ocata i'd get that placement thing installed that keeps spamming the logs | 17:35 |
| markguz | i originally followed the guide for configuring placement in ocata, but ran into problems when trying to do the create cell bit | 17:36 |
| jaypipes | markguz: right, since that spam is legit warning you about a required component :) | 17:36 |
| markguz | yeah... but where's the newton configure guide? | 17:37 |
| jaypipes | markguz: when you say "followed the guide", can you point us to the exact guide you are using please? | 17:37 |
| jaypipes | markguz: we need to know if this is RDO docs or upstream docs somewhere you are following :) | 17:37 |
| markguz | rdo netwon -> ocata docs | 17:38 |
| markguz | https://www.rdoproject.org/install/upgrading-rdo-3/ | 17:38 |
| markguz | seems like all the online docs are for ocata | 17:38 |
| markguz | as the nova-manage in netwon does not have the create_cell option | 17:39 |
| jaypipes | markguz: k, lemme review (FYI, create_cell doesn't have anything to do with placement) | 17:39 |
| markguz | why do all the docs say you need it then? or ami in the wrong chat group? | 17:39 |
| mriedem | cells v2 and placement are both required starting in ocata | 17:40 |
| mriedem | separate required thigns | 17:40 |
| mriedem | *things | 17:40 |
| markguz | hmm... i see | 17:40 |
| markguz | the docs seem to conflate it | 17:40 |
| markguz | IMHO | 17:40 |
| jaypipes | markguz: no prob. I was just making sure I understood what you are running into. | 17:40 |
| mriedem | https://docs.openstack.org/ocata/install-guide-ubuntu/nova.html | 17:41 |
| jaypipes | markguz: and it's helpful to be clear/explicit about *which* docs you refer to, since, as I mentioned above, there's docs from distros and there's docs from the upstream projects. | 17:41 |
| markguz | jaypipes: i wanted the official openstack docs for placement/cells but they don't seem to exist? | 17:42 |
| mriedem | https://docs.openstack.org/nova/ocata/man/nova-manage.html#nova-cells-v2 | 17:42 |
| mriedem | the install guide for ocata for ubuntu is linked above | 17:42 |
| markguz | mriedem: ocata not newton | 17:42 |
| jaypipes | markguz: ok, so to be clear, you are getting to step 14 of the "Upgrading the Compute Service (nova) on Controller Nodes" in the RDO upgrade docs and running into an error. Correct? | 17:42 |
| mriedem | the newton install guide is no longer hosted from what i can tel | 17:42 |
| mriedem | oh nvm https://docs.openstack.org/newton/install-guide-rdo/ | 17:42 |
| jaypipes | mriedem: this is upgrade newton -> ocata IIUC | 17:42 |
| mriedem | https://docs.openstack.org/newton/install-guide-rdo/nova.html | 17:43 |
| mriedem | https://docs.openstack.org/nova/ocata/placement.html#upgrade-notes | 17:43 |
| markguz | mriedem: yes but they don't talk about placement/cells_v2 | 17:43 |
| mriedem | https://docs.openstack.org/nova/ocata/cells.html#upgrade-minimal | 17:43 |
| mriedem | see above | 17:43 |
| mriedem | because placement and cells v2 were optional in newton | 17:44 |
| mriedem | required in ocata | 17:44 |
| mriedem | and the install guide isn't about upgrades, it's about base install | 17:44 |
| markguz | ic | 17:44 |
| markguz | hmm, then i did manage to install it. | 17:44 |
| mriedem | here is the deploy guide for placement from ocata https://docs.openstack.org/nova/ocata/placement.html#deployment | 17:44 |
| mriedem | presumably that is similar to what's in the install guide for ocata | 17:44 |
| markguz | mriedem: so after running simple cell setup and discover hosts i can't seem to delete or create instances | 17:45 |
| markguz | i just get exceed DISK_GB errors in the placement API | 17:45 |
| *** bhagyashri_s has joined #openstack-placement | 17:45 | |
| mriedem | did you create a cell0 mapping and db, and run the api_db sync | 17:46 |
| markguz | mriedem: yep | 17:46 |
| mriedem | is the cell1 cell mapping in the api db pointed at the correct db connection URL? | 17:47 |
| mriedem | with the correct credentials? | 17:47 |
| markguz | yes | 17:47 |
| mriedem | are the computes all configured to report inventory to placement? | 17:47 |
| mriedem | actually, do this first: nova-status upgrade check | 17:48 |
| mriedem | from your control node | 17:48 |
| mriedem | if that fails for any reason, you have to resolve those issues first | 17:48 |
| *** bhagyashris_ has quit IRC | 17:48 | |
| markguz | -bash: nova-status: command not found | 17:49 |
| markguz | is that part of Newton or Ocata? | 17:49 |
| mriedem | the rdo package probably didn't install that binary | 17:49 |
| mriedem | ocata | 17:49 |
| mriedem | nova-status is new in ocata | 17:49 |
| mriedem | you could run it from a venv | 17:49 |
| markguz | i haven;t installed ocata yet i was preparing for it | 17:49 |
| mriedem | oh ok | 17:50 |
| mriedem | well, you could still install ocata nova in a venv and run the CLI from there if you wanted | 17:50 |
| markguz | i didn't realise installing placement in newton would break it so bad :-( | 17:50 |
| markguz | i keep getting this in the placement logs now | 17:51 |
| markguz | Over capacity for DISK_GB on resource provider | 17:51 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262 | 17:51 |
| markguz | but i don't use local disks so i don't understand why it's saying that | 17:51 |
| mriedem | in newton we're just reporting inventory to placement, the scheduler isn't using it in newton (i don't think, it's been awhile) | 17:52 |
| jaypipes | mriedem: that's correct. | 17:53 |
| jaypipes | markguz: log entries != your installation is broken. | 17:53 |
| markguz | jaypipes: well duh, but couple that with the not being able deploy or delete instances any longer i put 2 and 2 together | 17:54 |
| cdent | markguz: have you pasted those logs anywhere we can see them yet? | 17:54 |
| cdent | logs nova-scheduler would be good too | 17:54 |
| mriedem | your server create issue is likely something with cells v2 setup | 17:54 |
| mriedem | you should have a cell0 db and mapping, and a cell1 mapping which maps to your existing 'nova' db | 17:55 |
| markguz | http://paste.openstack.org/show/723332/ | 17:55 |
| jaypipes | mriedem: but he hasn't actually upgraded to ocata yet... | 17:55 |
| mriedem | and the database_connection URLs on those have to be correct, there was a bug i seem to remember in newton or ocata where simple_cell_setup would default the cell0 db name if you didn't provide one, which could have been wrong | 17:55 |
| cdent | markguz: are you only upgrading placement and nothing else, or have you cascaded a bunch of other packages when doing that? Is placement on its own host? | 17:55 |
| mriedem | sure, but you can deploy cells v2 in newton | 17:56 |
| markguz | yes | 17:56 |
| markguz | cdent: yes | 17:56 |
| markguz | cdent: only placement and it's on it's own host | 17:56 |
| cdent | thanks | 17:56 |
| * cdent looks at logs | 17:56 | |
| mriedem | markguz: those look like placement-api logs | 17:56 |
| cdent | we also need nova-scheduler logs | 17:57 |
| mriedem | need to see where the server create failure happens, if it's a 500 on POST /servers then the failure is in the nova-api logs | 17:57 |
| cdent | placement believes that the inventory for the compute node is too low to accept allocations, but since nova-scheduler is not using placement to schedule, that error should be a non-issue | 17:58 |
| cdent | however, if the nova-compute is reporting the wrong amount of disk to the scheduler (for whatever reason) you'll see it in the scheduler's logs | 17:59 |
| cdent | (when no available hosts are found) | 17:59 |
| jaypipes | mriedem: well, those logs are clearly saying that *something* is using 239728 of DISK_GB on that compute node out of a total capacity of 247799. I'd be curious to see what the value of compute_nodes.local_gb is for the compute node in question. | 17:59 |
| jaypipes | yes, what cdent said. | 17:59 |
| markguz | jaypipes: we use ceph rbd and not local disk for nova instances | 18:00 |
| jaypipes | markguz: that doesn't matter... | 18:00 |
| mriedem | doesn't matter, nova reports it as a single pool | 18:00 |
| mriedem | like it's local to the compute | 18:00 |
| jaypipes | markguz: it's about what the compute_nodes.local_gb value is (I know, that sucks, but it is what it is) | 18:00 |
| mriedem | that's not new with placement | 18:00 |
| mriedem | would have been a problem for you in newton also | 18:00 |
| jaypipes | ya | 18:01 |
| mriedem | and placement eventually allows us to model that inventory differently to fix that issue | 18:01 |
| cdent | merged today! | 18:01 |
| markguz | yeah i'm trying to figure out which instance that is, does the resource id map to the hypervisor uuid? | 18:01 |
| mriedem | cdent: that's if you setup the shared disk aggregate manually right? | 18:01 |
| cdent | yes | 18:01 |
| markguz | not instance i mean compute node | 18:01 |
| mriedem | markguz: nova hypervisor-show should show you those details | 18:01 |
| cdent | mriedem: but still seems like a huge milestone | 18:02 |
| cdent | brb | 18:02 |
| mriedem | and: nova hypervisor-servers | 18:02 |
| mriedem | shows the servers running on a given host | 18:02 |
| markguz | i guess i must have fixed it by accident.. as i can now spin up images. but the previous attempts are still stuck. | 18:03 |
| mriedem | stuck? | 18:04 |
| mriedem | delete those ERROR state instances that failed to boot | 18:04 |
| markguz | just stuck schediling | 18:04 |
| markguz | not in ERROR | 18:04 |
| markguz | just stuck saying "scheduling" | 18:04 |
| mriedem | ok that's probably a cells v2 issue then | 18:04 |
| mriedem | those should be in ERROR state in the cell0 db | 18:04 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 18:04 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 18:04 |
| openstackgerrit | Brianna Poulos proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 18:04 |
| markguz | hmm. ok. well i go back in a poke around at that. and see whats what | 18:05 |
| mriedem | check your conductor logs | 18:05 |
| mriedem | scheduler would raise NoValidHost which conductor should handle and 'bury' the instance in the cell0 db in error state | 18:05 |
| mriedem | if cell0 isn't available or not configured correctly, there would be problems | 18:05 |
| mriedem | select * from nova_api.cell_mappings; | 18:06 |
| mriedem | to see your cell mappings and verify they are pointing at the correct dbs | 18:06 |
| markguz | mriedem: k. thanks for your help. and thanks for clarifying the difference between placement and cells. | 18:06 |
| mriedem | np | 18:08 |
| * mriedem shares part of his thanks pie with jaypipes and cdent | 18:09 | |
| cdent | mmmm pie | 18:09 |
| cdent | I feel like we haven't really got to the bottom of what's going on, but if it is working, cool. Still seems like at least one hypervisor is reporting incorrect disk inventory (at least to placement, but maybe also to nova-scheduler) | 18:10 |
| cdent | In other news, omg, what a festival of merge coflicts we've had today | 18:10 |
| efried | That's what you get when so many things merge. | 18:17 |
| efried | not a terrible problem to have. | 18:17 |
| *** mriedem has quit IRC | 18:19 | |
| *** mriedem has joined #openstack-placement | 18:21 | |
| mriedem | cdent: makes me wonder, i bet we don't have anything in the docs about how to model your computes as using shared storage | 18:23 |
| openstackgerrit | Simon Dodsley proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530 | 18:43 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 18:45 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 18:45 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 18:45 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 18:45 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 18:45 |
| *** e0ne has quit IRC | 18:49 | |
| cdent | efried: yeah, use of "festival" intentional | 18:52 |
| cdent | mriedem: you mean in the old (prior to today) way? | 18:52 |
| efried | cdent: working up a response to dansmith/jaypipes on restricting reshapes to compute startup fyi | 18:53 |
| efried | cdent: ...done. Enjoy. | 18:54 |
| cdent | i shall try | 18:54 |
| efried | cdent: db separation stack - is that out of merge conflict at this point? | 18:54 |
| cdent | a few minutes ago it was | 18:55 |
| efried | :) | 18:55 |
| mriedem | cdent: no i mean in the new way | 18:57 |
| efried | mriedem: it's only libvirt that's "fixed" atm fyi | 18:59 |
| cdent | I've (perhaps incorrectly) been assuming that at least in part we were hoping to have a few minutes to let things bed in, and also because tetsuro's fixes for shared provides were proceeding asynchronously (I think?) | 18:59 |
| mriedem | ok i didn't think it needed to be per-driver since it's a generic setup | 19:00 |
| mriedem | and ok if we want to let it bake | 19:00 |
| cdent | I spoke with my vmware colleagues about it earlier today, they are keen to add is to the vmwareapi driver soon (for storage resources that are shared between vcenters, which is apparently a thing) | 19:00 |
| mriedem | it is a long-standing known issue though so if it's finally resolved, it's a big thing and we should have some accompanying docs for how to configure your computes to model shared storage | 19:00 |
| cdent | yeah, totally agree | 19:01 |
| efried | mriedem: It's per driver at least at the moment because the driver is responsible for reporting its inventory via update_provider_tree. | 19:04 |
| efried | The mechanism for manipulating the provider tree is going to differ from hypervisor to hypervisor. | 19:05 |
| efried | In libvirt's case, you have to create the shared provider manually and wrap an aggregate around it and the computes it shares with. | 19:06 |
| efried | In PowerVM, we're going to do that automatically in the driver. | 19:06 |
| efried | Don't know what vcenter is planning. | 19:06 |
| cdent | Undefined at this point, but likely something similar since the info will be available from vsphere | 19:10 |
| efried | cdent: +2s up the stack on db separation. Reckon gibi will +A those tomorrow. | 19:12 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | 19:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | 19:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 19:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 19:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324 | 19:13 |
| cdent | efried: flarglebargle, keep missing those commit message nits in the merge conflict fray | 19:15 |
| efried | cdent: figgered, no biggie. | 19:15 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324 | 19:28 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: DNM: Depends on intel-nfv-ci-tests to test NUMA LM https://review.openstack.org/574872 | 19:53 |
| openstackgerrit | Merged openstack/nova master: Refactor libvirt get_memory_used_mb() https://review.openstack.org/571030 | 20:00 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398 | 20:34 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: libvirt live migration: fit NUMA topology on dest https://review.openstack.org/567242 | 20:35 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723 | 20:35 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: DNM: Depends on intel-nfv-ci-tests to test NUMA LM https://review.openstack.org/574872 | 20:35 |
| *** ttsiouts has joined #openstack-placement | 20:51 | |
| *** smcginnis[m] has quit IRC | 20:53 | |
| *** ttsiouts has quit IRC | 20:53 | |
| *** ttsiouts has joined #openstack-placement | 20:54 | |
| *** smcginnis[m] has joined #openstack-placement | 21:07 | |
| *** ttsiouts has quit IRC | 21:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumb trusted_certs through the compute service https://review.openstack.org/561262 | 21:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 21:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 21:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 21:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add trusted certs to feature support matrix docs https://review.openstack.org/574890 | 21:16 |
| *** nicolasbock has quit IRC | 21:51 | |
| *** edmondsw has quit IRC | 22:26 | |
| *** edmondsw has joined #openstack-placement | 22:26 | |
| efried | jaypipes: I'm caught up on the consumer gen series, I think. | 22:26 |
| *** edmondsw has quit IRC | 22:31 | |
| *** ttsiouts has joined #openstack-placement | 22:37 | |
| *** ttsiouts has quit IRC | 22:39 | |
| *** ttsiouts has joined #openstack-placement | 22:40 | |
| *** ttsiouts has quit IRC | 22:42 | |
| *** ttsiouts has joined #openstack-placement | 22:43 | |
| *** ttsiouts has quit IRC | 22:44 | |
| *** ttsiouts has joined #openstack-placement | 22:45 | |
| mriedem | efried: i plan on hitting the rest of that tomorrow morning | 22:45 |
| mriedem | should be able to get it merged tomorrow | 22:45 |
| efried | mriedem: Well, the next patch in the series anyway. The main one has problems. | 22:45 |
| *** ttsiouts has quit IRC | 22:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove max_size parameter from fake_libvirt_utils.fetch_*image methods https://review.openstack.org/574911 | 22:47 |
| *** ttsiouts has joined #openstack-placement | 22:47 | |
| *** ttsiouts has quit IRC | 22:49 | |
| *** ttsiouts has joined #openstack-placement | 22:49 | |
| *** cdent has quit IRC | 22:51 | |
| *** ttsiouts has quit IRC | 22:51 | |
| *** ttsiouts has joined #openstack-placement | 22:52 | |
| *** ttsiouts has quit IRC | 22:54 | |
| *** ttsiouts has joined #openstack-placement | 22:55 | |
| *** ttsiouts has quit IRC | 22:57 | |
| *** ttsiouts has joined #openstack-placement | 22:58 | |
| *** ttsiouts has quit IRC | 23:00 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add enhanced KVM storage QoS quotas https://review.openstack.org/558530 | 23:00 |
| *** ttsiouts has joined #openstack-placement | 23:00 | |
| *** ttsiouts_ has joined #openstack-placement | 23:03 | |
| *** ttsiouts has quit IRC | 23:03 | |
| *** ttsiouts_ has quit IRC | 23:05 | |
| *** ttsiouts has joined #openstack-placement | 23:05 | |
| *** ttsiouts has quit IRC | 23:07 | |
| *** ttsiouts has joined #openstack-placement | 23:07 | |
| *** ttsiouts has quit IRC | 23:10 | |
| *** ttsiouts_ has joined #openstack-placement | 23:10 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: expand on various bdm parameters https://review.openstack.org/574805 | 23:10 |
| *** ttsiouts_ has quit IRC | 23:12 | |
| *** ttsiouts has joined #openstack-placement | 23:12 | |
| *** ttsiouts has quit IRC | 23:13 | |
| *** ttsiouts has joined #openstack-placement | 23:13 | |
| *** ttsiouts has quit IRC | 23:14 | |
| *** ttsiouts has joined #openstack-placement | 23:14 | |
| *** smcginnis[m] has quit IRC | 23:15 | |
| *** ttsiouts has quit IRC | 23:16 | |
| *** ttsiouts has joined #openstack-placement | 23:16 | |
| *** ttsiouts has quit IRC | 23:18 | |
| *** ttsiouts has joined #openstack-placement | 23:19 | |
| *** ttsiouts has quit IRC | 23:21 | |
| *** ttsiouts has joined #openstack-placement | 23:21 | |
| *** ttsiouts has quit IRC | 23:22 | |
| *** ttsiouts has joined #openstack-placement | 23:23 | |
| *** ttsiouts has quit IRC | 23:25 | |
| *** ttsiouts has joined #openstack-placement | 23:26 | |
| *** ttsiouts has quit IRC | 23:28 | |
| *** ttsiouts_ has joined #openstack-placement | 23:28 | |
| *** ttsiouts_ has quit IRC | 23:30 | |
| *** ttsiouts has joined #openstack-placement | 23:30 | |
| *** ttsiouts has quit IRC | 23:32 | |
| *** ttsiouts has joined #openstack-placement | 23:33 | |
| *** smcginnis[m] has joined #openstack-placement | 23:33 | |
| *** ttsiouts has quit IRC | 23:35 | |
| *** ttsiouts has joined #openstack-placement | 23:35 | |
| *** takashin has joined #openstack-placement | 23:36 | |
| *** ttsiouts has quit IRC | 23:37 | |
| *** ttsiouts has joined #openstack-placement | 23:37 | |
| *** ttsiouts has quit IRC | 23:39 | |
| *** ttsiouts has joined #openstack-placement | 23:40 | |
| *** ttsiouts has quit IRC | 23:42 | |
| *** ttsiouts_ has joined #openstack-placement | 23:42 | |
| mriedem | jaypipes: btw, fyi on the heal_allocations change to fix missing/incomplete consumers https://review.openstack.org/#/c/574488/ - wip for now | 23:43 |
| mriedem | but didn't want to duplicate work | 23:44 |
| *** ttsiouts_ has quit IRC | 23:44 | |
| *** ttsiouts has joined #openstack-placement | 23:44 | |
| *** ttsiouts has quit IRC | 23:46 | |
| *** ttsiouts has joined #openstack-placement | 23:47 | |
| *** ttsiouts has quit IRC | 23:49 | |
| *** ttsiouts has joined #openstack-placement | 23:49 | |
| *** ttsiouts has quit IRC | 23:51 | |
| *** ttsiouts has joined #openstack-placement | 23:58 | |
| *** takashin has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!