| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-31 | |||
| 00:20:41 | openstackgerrit | Zhenyu Zheng proposed openstack/nova: Don't delete neutron port when attach failed https://review.openstack.org/403568 | |
| 00:34:43 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759 | |
| 00:39:03 | openstackgerrit | Matt Riedemann proposed openstack/nova: Fix tag attribute disappearing in 2.33 and 2.37 https://review.openstack.org/424759 | |
| 00:40:31 | openstackgerrit | Matt Riedemann proposed openstack/nova: Pass APIVersionRequest to extensions https://review.openstack.org/425876 | |
| 00:43:29 | mriedem | artom: i cleaned up the bottom change, +2 on that one now | |
| 00:43:32 | mriedem | dansmith: ^ | |
| 00:54:11 | mriedem | jesus is this normal http://logs.openstack.org/06/425806/8/gate/gate-tempest-dsvm-cells-ubuntu-xenial/344170a/logs/screen-n-api.txt.gz?level=TRACE ? | |
| 01:21:25 | artom | mriedem, ah, much thanks! | |
| 01:21:36 | artom | I got your review but was busy with offsprings | |
| 01:21:37 | mriedem | artom: i'll have the tempest patch up shortly | |
| 01:21:51 | artom | All sleeping now, but you got ahead of me ^_^ | |
| 01:21:54 | mriedem | i'll have to deal with offspring in a bit | |
| 01:22:12 | mriedem | dreaded reading homework, the bane of my existence | |
| 01:23:34 | artom | For what it's worth, I kept the single device_tags in parameters.yaml because, if we can't min/max/min on them, I saw no point in splitting them up | |
| 01:24:48 | mriedem | the point in splitting them up was to keep a bit more specific description for each parameter | |
| 01:25:01 | mriedem | so we didn't have to cram either/or type wording into a single description | |
| 01:25:24 | mriedem | https://review.openstack.org/#/c/426991/ is the tempest change | |
| 01:26:23 | artom | mriedem, hrmm, looks like I didn't completely grok it then | |
| 01:29:41 | artom | Oh, duh, I get the point of splitting them | |
| 01:33:01 | artom | mriedem, is there the same rc1 deadline on the novaclient bug? | |
| 01:33:22 | mriedem | artom: novaclient is frozen for ocata | |
| 01:33:49 | mriedem | we can backport https://review.openstack.org/#/c/424745/ though | |
| 01:33:51 | mriedem | later | |
| 01:34:30 | artom | Aha, right | |
| 01:35:27 | gmann | artom: nova patch looks good | |
| 01:35:55 | artom | gmann, wootness, thanks for following up :) | |
| 01:37:10 | gmann | mriedem: thanks for tempest tests, ll check today | |
| 01:40:38 | gmann | mriedem: on all_tenant policy thing - https://review.openstack.org/#/q/topic:bp/add-whitelist-for-server-list-filter-sort-parameters | |
| 01:41:24 | gmann | mriedem: if we are deferring that to pike i can update doc patch of sort/ filter (removing the policy bits) | |
| 01:41:42 | gmann | mriedem: we need to get release notes/docs for sort/filter in | |
| 01:41:45 | mriedem | gmann: yes we're deferring the policy changes to pike | |
| 01:41:48 | mriedem | so we can discuss at the PTG | |
| 01:42:04 | mriedem | and yeah i have the sort/filter reno patch in our ocata rc1 todo etherpad | |
| 01:42:11 | gmann | mriedem: cool. ll update doc one quickly | |
| 01:43:18 | openstackgerrit | Ghanshyam Mann proposed openstack/nova: Add release note and docs for filter/sort whitelist https://review.openstack.org/421760 | |
| 01:43:26 | gmann | mriedem: ^^ | |
| 01:47:03 | mriedem | gmann: thanks. i probably won't get to that until tomorrow at this rate. | |
| 01:47:26 | gmann | mriedem: sure. | |
| 01:47:34 | mriedem | my wife has decided that simply because she has to get up at 4am, she wants to get some sleep and therefore i need to finish the parenting duties for the evening. | |
| 01:48:14 | gmann | mriedem: cool. spend some good time with your kids | |
| 01:48:59 | gmann | i have oomichi in japan today. ll catch him too :) | |
| 01:49:53 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts https://review.openstack.org/421420 | |
| 01:49:54 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 delete_cell https://review.openstack.org/421459 | |
| 01:49:54 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance https://review.openstack.org/421436 | |
| 01:49:55 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 list_cells https://review.openstack.org/421458 | |
| 01:49:55 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts https://review.openstack.org/421442 | |
| 01:49:56 | openstackgerrit | Diana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell https://review.openstack.org/421444 | |
| 01:53:04 | diana_clarke | Now I need to switch to a GitHub workflow. I've grown to really like Gerrit. | |
| 01:55:33 | artom | diana_clarke, gerrithub? | |
| 01:56:09 | diana_clarke | artom: is that a thing... off to google it | |
| 01:59:17 | melwitt | mriedem: I just looked at the cells v1 job "Failed to get metadata for IP 10.1.3.255: no metadata" in n-api logs and I think it's normal in that, the error means it couldn't lookup a FixedIP by address. and in cells v1 fixed_ips are populated only in the child cell database, so any reads from the api cell will yield not found | |
| 02:06:06 | mriedem | melwitt: should that be an ERROR though? | |
| 02:06:15 | mriedem | plus, has that changed recently? | |
| 02:06:18 | mriedem | i don't remember seeing that before | |
| 02:06:24 | melwitt | for non cells v1, yes | |
| 02:06:44 | melwitt | and agreed I don't remember seeing it repeatedly. I thought it used to only show it once. so I'm not sure what changed | |
| 02:10:02 | openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: rbd: Remove unnecessary 'encode' calls https://review.openstack.org/412356 | |
| 03:06:55 | mriedem | so, bauzas' scheduler change has failed in the gate twice in the cells v1 job on the volume swap test... | |
| 03:07:25 | mriedem | and https://review.openstack.org/#/c/399735/ merged today | |
| 03:08:17 | mriedem | and we don't run the cells v1 job on the check queue for devstack, which re-enabled the test | |
| 03:08:36 | mriedem | https://review.openstack.org/#/c/374373/ | |
| 03:08:44 | mriedem | but i'm pretty sure that just introduced some kind of weird race failure | |
| 03:08:46 | mriedem | lyarwood: ^ | |
| 03:11:24 | mriedem | mdbooth: ^ | |
| 03:12:50 | mriedem | looks like it's also killing the ceph job | |
| 03:16:15 | mriedem | i'm assuming this is probably just something to do with cells v1 not supporting the swap volume API | |
| 03:17:34 | mriedem | https://bugs.launchpad.net/nova/+bug/1660511/ | |
| 03:17:35 | openstack | Launchpad bug 1660511 in OpenStack Compute (nova) "TestVolumeSwap.test_volume_swap fails in cells v1 job waiting for first volume to detach" [High,Confirmed] | |
| 03:18:39 | mriedem | melwitt: if cells v1 doesn't support the swap volume API, shouldn't it fail in some kind of spectacular way? | |
| 03:19:28 | melwitt | mriedem: cells v1 non support usually fails in subtle crappy ways, in my experience | |
| 03:20:27 | mriedem | yeah that seems to be the case here | |
| 03:20:45 | mriedem | i was looking at the requests in a failed job earlier and at some point it looked like something just didn't happen, | |
| 03:20:49 | mriedem | like one of the requests just stopped | |
| 03:20:52 | melwitt | is this a new test, i.e. something that needs to be added to the blacklist? | |
| 03:21:01 | mriedem | it's a newly enabled test as of today | |
| 03:21:03 | mriedem | based on tempest config | |
| 03:21:04 | melwitt | ah | |
| 03:21:08 | mriedem | so i'm going to disable it for cells v1 in devstack | |
| 03:21:18 | mriedem | we could also blacklist it in nova, | |
| 03:21:30 | mriedem | which would probably actually make this go faster since we can fast approve it and not wait for devstack cores | |
| 03:21:33 | mriedem | mtreinish: are you around? | |
| 03:21:40 | melwitt | oh, disable the tempest conf option if cells v1 in devstack | |
| 03:22:26 | mriedem | yeah | |
| 03:22:28 | mriedem | i'm doing that now | |
| 03:24:32 | mriedem | https://review.openstack.org/427009 | |
| 03:26:10 | mriedem | if i don't get a quick ack from the cores i've pinged, i'll push up a blacklist patch in nova and just move forward that way | |
| 03:27:06 | melwitt | okay | |
| 03:30:19 | openstackgerrit | Matt Riedemann proposed openstack/nova: Blacklist test_volume_swap from cells v1 testing https://review.openstack.org/427013 | |
| 04:48:39 | abhishekk | is there any provision in nova to restrict users from addind certain metadata to server? | |
| 04:49:55 | mriedem | here is the not-so-subtle error http://logs.openstack.org/06/425806/8/check/gate-tempest-dsvm-cells-ubuntu-xenial/ab0bba2/logs/screen-n-cell-child.txt.gz?level=TRACE#_2017-01-31_01_25_10_115 | |
| 06:59:17 | openstackgerrit | Dinesh Bhor proposed openstack/nova: Remove unused exceptions from nova.exception https://review.openstack.org/427041 | |
| 08:06:18 | openstackgerrit | Balazs Gibizer proposed openstack/nova: reno for notification-transformation-ocata https://review.openstack.org/426810 | |
| 08:54:12 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: List/show all server migration types https://review.openstack.org/407237 | |
| 08:56:37 | tdurakov | pkoniszewski:hi, left several comments on liv-mig claim patch, could discuss them on irc if you want | |
| 09:08:51 | openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings https://review.openstack.org/407364 | |
| 09:10:34 | openstackgerrit | sahid proposed openstack/nova-specs: Libvirt driver emulator threads placement policy https://review.openstack.org/427066 | |
| 09:26:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | |
| 09:42:15 | openstackgerrit | Jordan Pittier proposed openstack/nova: Fix unspecified bahavior on GET /servers/detail?tenant_id=X as admin https://review.openstack.org/426259 | |
| 09:45:19 | abhishekk | hi all, is there any provision in nova to restrict users from adding certain metadata to server? | |
| 09:59:01 | openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | |