| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-29 | |||
| 11:14:10 | gmann | alex_xu: and 3rd is to merge the URl mapping and controller bits | |
| 11:15:14 | alex_xu | gmann: yes, 3rd is to use the plain route list instead of stevedore https://review.openstack.org/#/c/445864/7/nova/api/openstack/compute/routes.py@119 | |
| 11:15:41 | gmann | alex_xu: yea. | |
| 11:16:14 | gmann | alex_xu: so mainly 7 and 8 also kind of same as 6 but you separated for server create | |
| 11:16:22 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |
| 11:16:27 | alex_xu | gmann: yea | |
| 11:16:45 | gmann | alex_xu: so you want 6, 7 and 8 in last ? i thought first these and then stevedore things | |
| 11:17:56 | alex_xu | gmann: yes, 6,7,8 will spend a lot of time for us, if we do those first, I feel we need a long time to reach the point of remove stevedore | |
| 11:18:00 | openstackgerrit | Jaivish Kothari(janonymous) proposed openstack/os-vif master: Argument should have 2 params https://review.openstack.org/451338 | |
| 11:18:18 | alex_xu | gmann: so I try to remove the stevedore first, then let us cleanup 6,7,8 slowly | |
| 11:18:37 | alex_xu | gmann: I hope 3rd is easier than 6,7,8 | |
| 11:19:21 | gmann | alex_xu: i like 6.7.8 easy :) but yea those can take much time so its good to remove the small bits early | |
| 11:20:00 | gmann | alex_xu: i can help on 3rd too. let's see we can touch 6,7,8 also in Pike at least little bit | |
| 11:20:30 | alex_xu | gmann: thanks! | |
| 11:27:07 | openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230 | |
| 11:28:52 | alex_xu | gmann: really hope can merge those json-schema, due to this ugly code https://review.openstack.org/#/c/451194/2/nova/api/openstack/compute/servers.py@733 | |
| 11:28:53 | openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230 | |
| 11:34:13 | openstackgerrit | Alex Xu proposed openstack/nova master: Remove the rebuild extension help methods https://review.openstack.org/451346 | |
| 12:11:49 | openstackgerrit | jichenjc proposed openstack/nova master: add usage doc for resource provider https://review.openstack.org/450105 | |
| 12:29:43 | openstackgerrit | jichenjc proposed openstack/nova master: Remove the extension check for os-quota-class-sets https://review.openstack.org/451193 | |
| 12:35:32 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/451043 | |
| 12:45:38 | alex_xu | nova api meeting is in 15 mins at #openstack-meeting-4 | |
| 12:56:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove get_nw_info_for_instance from compute.utils https://review.openstack.org/451387 | |
| 12:56:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: remove ugly local import https://review.openstack.org/450787 | |
| 12:58:01 | gmann | alex_xu: yea scheduler_hints is really on wrong place on request | |
| 12:58:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'agent' commands https://review.openstack.org/450795 | |
| 12:58:07 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate 'shell' commands https://review.openstack.org/450794 | |
| 12:59:27 | alex_xu | gmann: yea | |
| 13:00:26 | alex_xu | nova api meeting is running | |
| 13:34:44 | dansmith | sdague: I get this locally when running functional-py35 .. any ideas what I'm doing wrong? https://pastebin.com/28hkZ2TG | |
| 13:35:25 | dansmith | or, hmm, that stacktrace shows some 27 and some 35 in the same process I guess | |
| 13:38:38 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/451043 | |
| 13:40:31 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049 | |
| 13:43:40 | bauzas | dansmith: yup, I looked at your errors, seeems unrelated to your tests, isn't it? | |
| 13:43:51 | bauzas | your s/tests/change | |
| 13:43:59 | dansmith | bauzas: seems like it, but I wanted to try to reproduce it locally to figure it out | |
| 13:44:15 | dansmith | I figured if everyone was hitting that there would be screaming all around, so.. | |
| 13:44:17 | bauzas | one is a connection timeout AFAIK | |
| 13:44:24 | bauzas | AFAICS rather | |
| 13:44:39 | bauzas | the other says "sorry, we don't expect this but that" | |
| 13:44:48 | dansmith | I know | |
| 13:44:51 | bauzas | and both are very similar so /o\ | |
| 13:44:55 | dansmith | have you seen it elsewhere/ | |
| 13:45:16 | bauzas | nope, but honestly haven't checked logstash yet | |
| 13:45:27 | bauzas | so possibly some new cool race conditions | |
| 14:04:11 | frickler | repeating my question from #openstack here: when I have attached a volume to a server, how can I determine afterwards whether delete_on_termination is true or false? | |
| 14:06:00 | dansmith | bauzas: tox from py35 lets me run those tests, by the way | |
| 14:07:20 | bauzas | dansmith: okay, so it's either a CI issue or a race | |
| 14:07:45 | dansmith | bauzas: no, I mean I can run functional-3.5 as a target | |
| 14:07:54 | dansmith | I haven't gotten to the two specific tests yet | |
| 14:07:58 | bauzas | ah ok | |
| 14:12:59 | sdague | dansmith: the only thing I can imagine is virtualenv on your base computer is really old? | |
| 14:13:28 | dansmith | sdague: just uninstalling tox and re-installing with python3 seems to have fixed it | |
| 14:13:33 | sdague | dansmith: ah | |
| 14:13:38 | sdague | any idea how old old tox was? | |
| 14:13:45 | sdague | is that in your uninstall logs | |
| 14:14:08 | dansmith | it was from pip before, probably fairly old, not sure I'd have logs of that kind of uninstall | |
| 14:14:13 | dansmith | and it's long since gone off my screen | |
| 14:14:52 | sdague | dansmith: ok, because if there is a new min we need it would be good to know, we can do that in tox.ini, then it will at least freak out if things aren't new enough with an understood message | |
| 14:15:18 | dansmith | sdague: yeah, wish I had thought of it | |
| 14:15:24 | dansmith | I don't think I have pip logs of that anywhere | |
| 14:17:16 | sdague | I honestly expect we probably need 2.3.0 as a minimum | |
| 14:17:21 | sdague | there is a 2.2ish fix around env | |
| 14:17:49 | dansmith | bauzas: and yes, those tests all pass locally :) | |
| 14:18:03 | bauzas | logstash-ing it | |
| 14:18:13 | bauzas | I'm a bit rusty | |
| 14:18:42 | dansmith | sdague: well, the weirdness was that it was importing things from py27 and py35 on the system | |
| 14:18:44 | dansmith | but yeah | |
| 14:19:05 | sdague | yeh, but there was something about basepython and env clobbering each other | |
| 14:21:04 | rpodolyaka | frickler: it's in instance detailed info - http://paste.openstack.org/show/604665/ - i.e. GET /servers/ |
|
| 14:21:05 | dansmith | okay | |
| 14:24:53 | openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Convert all 'ip-link set' commands to pyroute2 https://review.openstack.org/451433 | |
| 14:27:02 | frickler | rpodolyaka: ah, great, the v2.41 is the trick, osc uses 2.1 by default and then it is missing. if I explicitly call for 2.41, I see the correct information. thx alot | |
| 14:27:39 | rpodolyaka | frickler: oops, that may be a red herring - that just what I get for `nova --debug show <uuid>` | |
| 14:27:59 | rpodolyaka | frickler: so just test it on 2.1, I | |
| 14:28:29 | rpodolyaka | *I'm not aware of any changes related to the info we return on attached volumes | |
| 14:28:34 | frickler | rpodolyaka: no, I confirmed that "openstack --os-compute-api-version=2.41 server show test1" shows the attribute, without setting the api version it doesn't | |
| 14:28:42 | frickler | rpodolyaka: was added in 2.16 it seems | |
| 14:28:49 | rpodolyaka | ah, ok | |
| 14:29:15 | frickler | makes me wonder why OSC doesn't use latest API by default | |
| 14:30:35 | rpodolyaka | frickler: neither does python-novaclient. Every new microversion is potentially backward-incompatible - you have to explicitly ask for it and make changes to the code (if necessary) | |
| 14:31:54 | openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 14:32:07 | sdague | rpodolyaka: that's not true, novaclient CLI uses latest by default | |
| 14:33:31 | rpodolyaka | sdague: hmm, won't it break novaclient if we remove field X from the output of some endpoint and novaclient expects it to be present? | |
| 14:33:54 | rpodolyaka | I thought we defaulted to the minimum | |
| 14:34:06 | rpodolyaka | and only ask for newer version if that's needed | |
| 14:34:26 | rpodolyaka | or if the command, let's say nova show now supports it (and so does the nova-api we talk to) | |
| 14:35:51 | efried | Did nova recently change what kind of threads the compute process uses? | |
| 14:36:12 | rpodolyaka | sdague: or by latest you mean latest supported by novaclient? not the latest returned by nova-api? | |
| 14:36:43 | sdague | rpodolyaka: we default to latest novaclient understands that the server supports | |
| 14:36:55 | rpodolyaka | ah, I see, that makes sense | |
| 14:36:59 | rpodolyaka | frickler: ^ | |
| 14:37:08 | rpodolyaka | frickler: sorry for misleading you :P | |
| 14:37:25 | sdague | efried: nothing that I know of, what kind of things are you running into? | |
| 14:37:55 | efried | sdague Got an open() hanging in a subthread - and it's hanging the entire compute process. | |
| 14:38:55 | sdague | efried: there was an eventlet bump not that long ago | |
| 14:38:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move 'floating_ips' opts into 'network' https://review.openstack.org/431039 | |
| 14:38:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename '[neutron] ovs_bridge' https://review.openstack.org/450266 | |
| 14:38:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add neutron.default_floating_pool https://review.openstack.org/431037 | |
| 14:38:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'default_floating_pool' https://review.openstack.org/431038 | |