| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-31 | |||
| 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 | |
| 10:01:48 | openstackgerrit | John Garbutt proposed openstack/nova: DNM: hack ironic with resource providers https://review.openstack.org/426296 | |
| 10:13:57 | johnthetubaguy | bauzas: are you around by any chance, and do you have the context around the ironic changes? | |
| 10:21:59 | bauzas | johnthetubaguy: I'm here yup | |
| 10:22:33 | johnthetubaguy | bauzas: I wondered if you could check my comments on this patch: https://review.openstack.org/#/c/404472 | |
| 10:22:41 | johnthetubaguy | not sure if I am talking nonsense or sense | |
| 10:23:49 | andymccr | morning all - im having a few issues with the placement service, was hoping for a bit of guidance. The first is that it seems you can't define which endpoint interface compute connects to placement on. (I put a PR in for that which worked for me but I may be way off the mark: https://review.openstack.org/#/c/426163/ ) | |
| 10:23:50 | openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.add_fixed_ip notification https://review.openstack.org/332876 | |
| 10:25:09 | andymccr | secondly, i get a 409 on the first PUT from the compute host, 200 on all GETs but then 400's on subsequent PUTs. The errors are vague though - I'm not too sure what steps I can take to debug that further. | |
| 10:29:44 | johnthetubaguy | andymccr: hmm, interesting | |
| 10:29:53 | johnthetubaguy | andymccr: the endpoint is picked using the keystone service catalog | |
| 10:30:02 | johnthetubaguy | (or should be) | |
| 10:30:34 | andymccr | johnthetubaguy: yeah it is, but it forces "public" afaict because the keystoneauth endpoint_filter doesnt include an interface option when passed by nova | |
| 10:30:34 | johnthetubaguy | andymccr: oh, the interface | |
| 10:30:46 | johnthetubaguy | should we really default to private? | |
| 10:31:05 | andymccr | my pr defaults it to public - so in essence, no change if you don't set it | |
| 10:31:33 | andymccr | internal works for my usecase, but not sure it'd work for everybody - so figured its best to not change things like that midway :) rather make them changeable. | |
| 10:32:04 | andymccr | i imagine in things like role gates/devstack it makes no difference since the endpoints are probably all the same regardless of internal/public | |
| 10:32:21 | johnthetubaguy | its an admin only API (at least by design) so its probably a mute point | |
| 10:33:01 | andymccr | if its admin only perhaps it should default to admin instead? | |
| 10:33:32 | andymccr | from my pov, i need it to not be public or else i have to do some munging when i create endpoints to ensure the public endpoint is created differently to all other public endpoints and uses the internal ip | |
| 10:34:24 | johnthetubaguy | oh, possibly, yeah, default to admin makes more snese | |
| 10:35:47 | johnthetubaguy | in theory non admin users would just get 403 errors I think, but its good to message that better | |
| 10:36:23 | johnthetubaguy | bauzas: cdent: do you have better context on the intended deployment stuff, for our OSA friends? | |
| 10:37:09 | bauzas | johnthetubaguy: sorry, I had an internal problem | |
| 10:37:29 | bauzas | and I'm very busy, so could you please explain me the issue ? | |
| 10:37:55 | johnthetubaguy | bauzas: service catalog, we default to accessing the "public" URL for placement | |
| 10:38:05 | johnthetubaguy | that seems odd, given its an admin only API | |
| 10:38:10 | andymccr | (and you can't override it afaict) | |
| 10:38:25 | bauzas | lemme verify something | |
| 10:38:47 | bauzas | I think nova just doesn't use the public field | |