| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-18 | |||
| 07:59:00 | openstackgerrit | Brin Zhang proposed openstack/nova master: Pass RequestContext to oslo_policy https://review.opendev.org/674038 | |
| 07:59:01 | openstackgerrit | Brin Zhang proposed openstack/nova master: Introduce scope_types in os-services https://review.opendev.org/645427 | |
| 07:59:01 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add test coverage of existing os-services policies https://review.opendev.org/669181 | |
| 07:59:02 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add policy deprecation fixture and Suppress warnings in tests https://review.opendev.org/676670 | |
| 07:59:02 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452 | |
| 08:01:08 | openstackgerrit | Brin Zhang proposed openstack/nova master: Pass RequestContext to oslo_policy https://review.opendev.org/674038 | |
| 08:03:27 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add test coverage of existing os-services policies https://review.opendev.org/669181 | |
| 08:03:34 | openstackgerrit | Brin Zhang proposed openstack/nova master: Introduce scope_types in os-services https://review.opendev.org/645427 | |
| 08:04:23 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add new default roles in os-services API policies https://review.opendev.org/648480 | |
| 08:07:23 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452 | |
| 08:30:56 | openstackgerrit | Brin Zhang proposed openstack/nova master: Add policy deprecation fixture and Suppress warnings in tests https://review.opendev.org/676670 | |
| 09:14:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor rebuild_instance https://review.opendev.org/688419 | |
| 09:14:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enable evacuation with qos ports https://review.opendev.org/688688 | |
| 09:14:14 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow evacuating server with port resource request https://review.opendev.org/688387 | |
| 09:55:14 | openstackgerrit | Matthew Booth proposed openstack/nova master: Unplug VIFs as part of cleanup of networks https://review.opendev.org/663382 | |
| 09:55:14 | openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/689278 | |
| 09:55:15 | openstackgerrit | Matthew Booth proposed openstack/nova master: Fix incorrect vm and task state after build failure race https://review.opendev.org/689388 | |
| 09:55:24 | mdbooth | stephenfin sean-k-mooney: ^^^ | |
| 10:00:13 | mdbooth | sean-k-mooney: The bottom of that stack is the new test base we discussed, btw, which also has a +1 from zuul. It would be useful to do the rounds on that one sooner rather than later. If you could rebase your patch on top of it so we can show it's useful that would be even better. | |
| 10:02:16 | sean-k-mooney | ya i can do that | |
| 10:40:19 | ygk_12345 | hi all | |
| 10:41:13 | ygk_12345 | I am going thru the nova code on github. Van anyone point me to the exact code which says the libvirt to use qemu-kvm binary to launch a new process for the vm ? | |
| 10:41:21 | ygk_12345 | *Can anyone | |
| 10:41:43 | openstackgerrit | Merged openstack/nova master: Add support for 'initenv' elements https://review.opendev.org/667975 | |
| 10:43:12 | openstackgerrit | Matthew Booth proposed openstack/nova master: Unplug VIFs as part of cleanup of networks https://review.opendev.org/663382 | |
| 10:47:10 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Include removal of ephemeral backing files in the image cache manager https://review.opendev.org/689422 | |
| 11:17:15 | artom | ygk_12345, you mean you want to see a os.exec('qemu') in the code? You won't find it | |
| 11:17:29 | artom | ygk_12345, that's in the libvirt code itself | |
| 11:17:40 | artom | ygk_12345, Nova just defines a domain with some XML | |
| 11:17:40 | ygk_12345 | @artom where exactly ? | |
| 11:17:55 | artom | ygk_12345, I dunno, use grep, or ask in #libvirt | |
| 11:18:01 | artom | But mostly grep | |
| 11:18:25 | cdent | "But mostly grep" is my mantra | |
| 11:18:43 | ygk_12345 | @artom I have installed libvirt-python and used recursive grep but could not find the qemu-kvm pattern | |
| 11:18:49 | artom | Mostly Grep is my rap name | |
| 11:19:06 | cdent | oh, that's a good one | |
| 11:19:07 | artom | ygk_12345, that's because those are just the Python bindings for libvirt | |
| 11:19:22 | artom | ygk_12345, you want libvirtd | |
| 11:21:21 | artom | We form wht Unix Krew with Oh Snap it's Sed, Aw man it's Awk, and Who Let Perl In? | |
| 11:21:26 | artom | *the Unix | |
| 11:24:31 | ygk_12345 | @artom things at the OS level are weird but interesting | |
| 11:26:09 | cdent | ygk_12345: out of curiosity, why are you looking for the exec call? are you trying to solve a problem, exploring, something else? | |
| 11:27:07 | ygk_12345 | @cdent I am trying to understand and trace the vm create event through nova code :) | |
| 11:27:36 | cdent | I think what artom said is the key bit: nova creates some xml and gives it to libvirt, libvirt does the rest | |
| 11:28:25 | ygk_12345 | @cdent so its libvirt API internally | |
| 11:28:57 | ygk_12345 | @cdent awesome | |
| 11:29:28 | ygk_12345 | @cdent u see no limits to curiosity until we hit the hardware | |
| 11:29:37 | cdent | :) | |
| 11:29:57 | cdent | plenty of room for curiosity on the hardware too | |
| 11:30:45 | ygk_12345 | @cdent appreciate Linus Torvalds for his first version of Linux | |
| 12:17:24 | ygk_12345 | HI all ? | |
| 12:17:26 | ygk_12345 | which wsgi framework is used in nova ? | |
| 12:19:42 | sean-k-mooney | we kind of have our own https://github.com/openstack/nova/blob/master/nova/api/wsgi.py | |
| 12:19:56 | ygk_12345 | ok | |
| 12:20:32 | ygk_12345 | does it have a standalone web server ? | |
| 12:20:46 | sean-k-mooney | it can run in several different modes | |
| 12:21:06 | sean-k-mooney | it can run on the built in python webserver with eventlets | |
| 12:21:27 | sean-k-mooney | or you can run the wsgi service via mod_wsgi or uwsgi | |
| 12:21:58 | ygk_12345 | as of now when we install nova-api, in which mode deso it erun ? | |
| 12:22:04 | ygk_12345 | *does it | |
| 12:22:21 | sean-k-mooney | the nova api console script uses https://github.com/openstack/nova/blob/master/nova/cmd/api.py | |
| 12:22:26 | sean-k-mooney | which uses eventlet | |
| 12:22:27 | sean-k-mooney | but | |
| 12:22:31 | sean-k-mooney | if you use devstack | |
| 12:22:36 | sean-k-mooney | or most installers | |
| 12:22:44 | sean-k-mooney | the nova-api service is not calling that script | |
| 12:22:58 | sean-k-mooney | its running the service under uwsgi i think | |
| 12:23:05 | ygk_12345 | oh | |
| 12:25:18 | sean-k-mooney | ygk_12345: cdent know more about this then i but we moved to testing with extrenal wsgi server implemenation a few release ago | |
| 12:25:43 | ygk_12345 | oh ok | |
| 13:42:28 | KeithMnemonic | mriedem Good day, what is the best was to get some reviews on the patch you submitted for me ? https://review.opendev.org/#/c/683008/ | |
| 13:43:29 | mriedem | KeithMnemonic: umm | |
| 13:43:45 | mriedem | you know we won't release that right? | |
| 13:43:58 | mriedem | since pike is in extended maintenance mode. we can merge stuff to pike but it won't be released. | |
| 13:44:08 | mriedem | so if you really need it you could probably just cherry pick it downstream | |
| 13:46:36 | KeithMnemonic | ok, i can do that, i was hoping it would get merged at least , is that not an option anymore even it if it is not released | |
| 13:47:13 | mriedem | it could be merged upstream but finding people to care about pike at this point upstream is hard, and that patch isn't exactly trivial | |
| 13:47:16 | mriedem | with all of the conflicts | |
| 13:48:16 | KeithMnemonic | ok let me try and cherry pick it locally | |
| 13:48:16 | dansmith | mriedem: api patch landed, can you kick this in? https://review.opendev.org/#/c/687141/ | |
| 13:49:42 | mriedem | dansmith: were you going to reply to andrey again before doing so? | |
| 13:50:16 | mriedem | he is a core so i feel sort of bad if i just overrule him without a reply | |
| 13:51:03 | dansmith | okay | |
| 13:54:16 | mdbooth | Oh, that's evil | |
| 13:54:52 | mdbooth | In python2, if your module is in nova.tests.functional and you 'import fixtures', you get nova.tests.functional.fixtures. In python3 you get 'fixtures'. | |
| 13:55:13 | mdbooth | Apart from renaming nova.tests.functional.fixtures, I'm not immediately sure wtf to do about that. | |
| 13:57:55 | mdbooth | Fun fact 2: emojis in review comments elicit a 500 from gerrit. | |
| 13:58:01 | sean-k-mooney | mdbooth: that to do with relitive imports and what happens if you have a module in the local director that alisas a module in your path | |
| 13:58:23 | mdbooth | sean-k-mooney: Any idea how to disable in py2 in a manner which doesn't break py3? | |
| 13:58:50 | sean-k-mooney | there might be a futre thing for this. absolute import or something like htat | |
| 13:59:21 | melwitt | KeithMnemonic, mriedem: I started reviewing that patch yesterday but not done yet. it's taking forever bc of all the conflicts. I'll finish reviewing today | |
| 13:59:36 | KeithMnemonic | thanks! | |
| 13:59:38 | sean-k-mooney | https://www.python.org/dev/peps/pep-0328/ | |
| 13:59:45 | sean-k-mooney | from __future__ import absolute_import | |
| 14:00:10 | sean-k-mooney | but that shoudl be enabled by default in py 2.7 | |
| 14:00:28 | mdbooth | Apparently not | |
| 14:05:36 | bauzas | mdbooth: to make it clear, nobody should just import any module like "import <module>" but rather doing "from <where> import <module>" | |
| 14:06:08 | bauzas | if I'm reviewing any change like this, I'd -1 it | |
| 14:09:00 | melwitt | mdbooth: there shouldn't be a module nova.tests.functional.fixtures module, there's one in nova.tests.fixtures though | |
| 14:09:11 | melwitt | I think that's how this hasn't come up before | |