| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-24 | |||
| 18:22:25 | openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 18:22:26 | openstackgerrit | Artom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 18:22:27 | openstackgerrit | Artom Lifshitz proposed openstack/nova: API support for tagged device attachment https://review.openstack.org/376021 | |
| 18:22:28 | openstackgerrit | Artom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 18:28:58 | openstackgerrit | STEW TY proposed openstack/nova: Transform instance.resize_confirm notification https://review.openstack.org/390654 | |
| 18:29:42 | openstackgerrit | Gábor Antal proposed openstack/nova: Remove mox in unit/cells/test_cells_manager.py https://review.openstack.org/399036 | |
| 18:36:34 | openstackgerrit | Artom Lifshitz proposed openstack/nova: Fix tag attribute disappearing in 2.32 and 2.37 https://review.openstack.org/424759 | |
| 19:17:11 | cdent | bauzas: in your (not so) copious free time, you might find this bug fix to the resource tracker interesting: https://review.openstack.org/#/c/424305/ | |
| 19:20:46 | openstackgerrit | Dan Smith proposed openstack/nova: WIP: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 19:20:47 | openstackgerrit | Dan Smith proposed openstack/nova: Move instance creation to conductor https://review.openstack.org/319379 | |
| 19:22:43 | dansmith | melwitt: johnthetubaguy: okay I think I can just nuke the extra validate, so I did ^ | |
| 19:24:48 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: DO NOT MERGE: experiment https://review.openstack.org/424809 | |
| 19:34:50 | artom | Well crap, how does a vif in network_info end up with a vlan? I'm working on the tempest test for https://review.openstack.org/#/c/405951/ and the only way I've found to set a vlan on a Neutron network is to use provider:segmentation_id, but that required provider:network_type to be vxlan and for me to be an admin | |
| 19:38:02 | openstackgerrit | Stephen Finucane proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell https://review.openstack.org/400874 | |
| 19:42:09 | dansmith | johnthetubaguy I think this is good to go now: https://review.openstack.org/#/c/410473/ | |
| 19:54:24 | openstackgerrit | Vladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata https://review.openstack.org/405951 | |
| 19:55:30 | openstackgerrit | STEW TY proposed openstack/nova: Transform instance.resize_revert notification https://review.openstack.org/396811 | |
| 20:02:48 | bauzas | dansmith: cdent: I could probably need some help for my grenade change | |
| 20:02:58 | bauzas | http://logs.openstack.org/30/424730/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/15ad17c/logs/grenade.sh.txt.gz#_2017-01-24_17_11_42_862 | |
| 20:03:53 | bauzas | cdent: we need to deploy the placement engine by grenade | |
| 20:04:10 | bauzas | given the newton devstack doesn't deploy yet the placement service | |
| 20:05:17 | dansmith | bauzas: you're not sourcing something that brings in the base variable I imagine | |
| 20:05:24 | cdent | $FILES is not defined, looks like it is usually defined by stack.sh | |
| 20:05:38 | bauzas | mmm ok | |
| 20:06:43 | dansmith | looks like in stack.sh | |
| 20:06:50 | dansmith | so define it in there before you call those methods | |
| 20:06:54 | dansmith | $TOP_DIR/files | |
| 20:08:16 | cdent | bauzas: see upgrade-tempest for an example | |
| 20:08:45 | cdent | bauzas: and/or projects/30_swift/upgrade.sh | |
| 20:08:52 | bauzas | cdent: roger | |
| 20:09:18 | cdent | looks like those use $TARGET_DEVSTACK_DIR/files | |
| 20:11:12 | bauzas | dansmith: I guess I can deploy grenade on my local devstack for testing ? | |
| 20:11:28 | dansmith | if you're really really lucky :) | |
| 20:11:29 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/424828 | |
| 20:12:25 | bauzas | dansmith: you mean it's difficult? | |
| 20:13:10 | dansmith | bauzas: I mean I often have issues | |
| 20:14:02 | cdent | bauzas: I agree with dansmith: I often have issues, and in addition it takes forever to reach those issues, at which point you throw your hands up in the air | |
| 20:14:18 | dansmith | yeah | |
| 20:14:34 | bauzas | okay, so let's hit the gate... | |
| 20:37:40 | amrith | andreykurilin, the patch to requirements repo has come down to trove via the req bot. I've dep'ed your change on that. Hope that's what you intended. | |
| 20:42:15 | andreykurilin | amrith: everything should work now in trove | |
| 20:42:19 | andreykurilin | without my patch | |
| 20:45:15 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type https://review.openstack.org/424256 | |
| 20:45:56 | amrith | andreykurilin, cool. in that case I'm going to announce my retirement and move to a beach somewhere where it is warm. | |
| 20:46:09 | amrith | :) | |
| 20:46:09 | andreykurilin | heh | |
| 20:46:21 | amrith | thanks andreykurilin I very much appreciate your help with this set of changes. | |
| 20:47:20 | andreykurilin | amrith: I'll return to them with new novaclient 7.1.0 :) but I'll try to do it before breaking gates | |
| 20:47:42 | amrith | OK, and I'll propose some of the tests that I mentioned this morning against head of nova | |
| 20:48:16 | andreykurilin | amrith: it will be cool, thanks | |
| 20:51:24 | openstackgerrit | STEW TY proposed openstack/nova: Transform instance.resize_revert notification https://review.openstack.org/396811 | |
| 21:01:33 | openstackgerrit | Dan Smith proposed openstack/nova: WIP: Collected release notes for Ocata CellsV2 https://review.openstack.org/424737 | |
| 21:01:33 | openstackgerrit | Dan Smith proposed openstack/nova: Move instance creation to conductor https://review.openstack.org/319379 | |
| 21:12:41 | mriedem | i hope my insurance covers a bad case of sand crack | |
| 21:15:18 | cdent | get some talcum powder on that ass | |
| 21:19:22 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: DO NOT MERGE: check OSC https://review.openstack.org/420941 | |
| 21:23:54 | mriedem | vladikr: so i was thinking about vlan vifs and tempest testing, we probably don't have any jobs that use vlan tags on ports do we? | |
| 21:29:44 | cdent | bauzas: your grenade run has just finished | |
| 21:29:56 | bauzas | cdent: yup, was following | |
| 21:30:06 | cdent | on the old side it looks like placement auth isn't in place: http://logs.openstack.org/30/424730/2/check/gate-grenade-dsvm-neutron-ubuntu-xenial/6ccea97/logs/old/screen-n-cpu.txt.gz#_2017-01-24_20_36_57_937 | |
| 21:30:47 | bauzas | cdent: that's for the full grenade job right? | |
| 21:31:02 | bauzas | cdent: if so, I think it's not a problem | |
| 21:31:13 | cdent | Oh, is it just changed for one of the jobs not all? | |
| 21:31:32 | bauzas | cdent: I was rather looking at the multinode job | |
| 21:31:51 | bauzas | cdent: I provided a from-newton script | |
| 21:32:04 | bauzas | cdent: so it's not changing the old side AFAICU | |
| 21:32:24 | cdent | ah, I follow now | |
| 21:33:03 | bauzas | cdent: that said, I'd like to see the old compute calling the new scheduler by grenade | |
| 21:33:13 | bauzas | cdent: so IIUC it's the multinode job | |
| 21:33:36 | bauzas | but maybe dansmith can insight? | |
| 21:34:11 | bauzas | I think I'm also starting the placement engine for any node, including a specific compute only | |
| 21:34:38 | cdent | bauzas: I've got to call it a night, but if there's something I can do to help out tomorrow morning leave me a note somewhere (email or here is fine) | |
| 21:34:52 | bauzas | cdent: I'm just rechecking the scheduler change | |
| 21:35:00 | bauzas | cdent: for seeing if it's now happy | |
| 21:35:03 | bauzas | cdent: sure | |
| 21:37:37 | dansmith | bauzas: from-$release only runs on the main node AFAK | |
| 21:38:28 | bauzas | dansmith: I replied to you on the ML thread about optional resources | |
| 21:38:38 | bauzas | dansmith: have you seen my other emails ? | |
| 21:38:45 | dansmith | yeah | |
| 21:39:01 | bauzas | and why I think it's understandable, I think asking that for Ocata seems difficult to me | |
| 21:39:17 | bauzas | the gate itself is even not running DiskFilter... | |
| 21:39:22 | bauzas | oops | |
| 21:39:26 | bauzas | CoreFilter I meant | |
| 21:39:57 | bauzas | and we also have an assertion that says amount can't be 0 - but we have a lot of flavors having root_gb=0 | |
| 21:40:51 | bauzas | so what should I ask for if a flavor is set to root_gb=0 for example ? | |
| 21:41:13 | bauzas | I mean, I totally understand jaypipes's opinion about why it's a wrong direction | |
| 21:41:24 | bauzas | I just think it's not something we should stop by Ocata | |
| 21:42:52 | cburgess | dansmith: If I have an instance object (returned from novaclient) is there an easy way to refresh the status and task_status from the DB? Or do I have to do a full server.get() again? | |
| 21:43:33 | dansmith | cburgess: you don't get instance objects back from novaclient, you get a dict.. so yeah, you have to .get() again | |
| 21:43:35 | cburgess | OK well not from the DB since clearly it has to make an API call since I'm using novaclient. I mean do you just get it again or is there some swanky built-in to refresh it? | |
| 21:43:40 | cburgess | Oh ok.. | |
| 21:44:18 | bauzas | dansmith: saw my points above ? | |
| 21:44:29 | cburgess | Hmm weird.. its got methods defined on it, but I get your point. I'll re-fetch it. | |
| 21:44:52 | dansmith | bauzas: yes, I've read your emails and the points above, I'm not sure what to say and I'm frantically trying to get other work done | |
| 21:44:55 | dansmith | bauzas: it's just MHO | |
| 21:45:23 | bauzas | dansmith: okay again, I don't disagree with both of you | |
| 21:45:30 | bauzas | dansmith: I'm just trying to be pragmatic | |
| 21:49:58 | vladikr | mriedem, sorry I was away.. Yes, I didn't find any jobs that uses vlans | |
| 21:50:59 | vladikr | mriedem, also, in order to test it, we need something that will set the {binding:{details:{vlan: X}}} | |
| 21:51:42 | vladikr | mriedem, which is not available for all vnic types | |