| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-24 | |||
| 14:11:21 | mriedem | takashin: ok yeah i remember talking about this in ocata so i thought there was a spec for it before | |
| 14:13:08 | takashin | mriedem: "List/Show all server migration types" was included in "Abort cold migration" spec at first. It is PS 22 in https://review.openstack.org/#/c/334732/ . | |
| 14:13:29 | takashin | mriedem: But I split it out to another spec. | |
| 14:16:59 | mriedem | EmilienM: does tripleo setup a keystone v3 endpoint in the service catalog? | |
| 14:17:09 | EmilienM | mriedem: I'm pretty sure yes | |
| 14:17:29 | mriedem | where in the tripleo log haystack can i find the endpoints be created? | |
| 14:17:33 | EmilienM | in our stackrc, we just don't force v3 because in the past ironic inspector didn't support v3 auth | |
| 14:17:52 | EmilienM | mriedem: we're in design sessio now with my team, I need to focus on the session | |
| 14:18:03 | EmilienM | mriedem: maybe we can meet face 2 face during the morning | |
| 14:19:14 | EmilienM | mriedem: http://logs.openstack.org/65/437365/3/check/gate-tripleo-ci-centos-7-nonha-multinode/bcbd374/logs/var/log/httpd/keystone_wsgi_main_access.txt.gz | |
| 14:19:28 | EmilienM | we enable v3 for sure | |
| 14:21:47 | sdague | yeh, looking at it | |
| 14:25:11 | sdague | EmilienM: I wonder if keystone isn't reacting as expected - https://developer.openstack.org/api-ref/identity/v3/index.html?expanded=list-projects-detail#list-projects | |
| 14:25:40 | sdague | actually - https://developer.openstack.org/api-ref/identity/v3/index.html?expanded=show-project-details-detail#show-project-details | |
| 14:26:01 | mriedem | takashin: did you talk to vmware or xenapi virt driver people about abort cold migration? | |
| 14:26:42 | mriedem | to see if that was possible for them? | |
| 14:28:02 | sfinucan | tonyb: You know requirements stuff. Any chance you could explain why we need to install from upper-constraints [1] when we have an auto-generated requirements.txt file in-tree? | |
| 14:28:04 | sfinucan | [1] https://github.com/openstack/nova/blob/master/tox.ini#L13 | |
| 14:30:31 | tonyb | sfinucan: u-c.txt is our statement about testing and compatibilty it specifies the exact version as opposed to requiremenst which give you a range | |
| 14:31:49 | openstackgerrit | Chris Friesen proposed openstack/nova-specs master: add spec for instance flavor API https://review.openstack.org/265282 | |
| 14:31:57 | sfinucan | tonyb: and (test-)requirements.txt? I bumped the version for a package in there but the upper-constraints seemed to take priority. Seems like we shouldn't have anything in there that can be found upper-constraints? | |
| 14:32:48 | danpawli1 | Can someone take a look on https://review.openstack.org/#/c/436953/ ? | |
| 14:32:59 | sfinucan | (the package in question was Sphinx. We ask for >=1.5.1 in test-requirements.txt but it's locked to 1.3.6 in upper-constraints) | |
| 14:33:40 | openstackgerrit | Manabu Ori proposed openstack/nova master: Not to skip schedulers when live migration is invoked from Horizon https://review.openstack.org/437921 | |
| 14:33:43 | tonyb | sfinucan: everything you have in *requirements.txt must come from global-requireemnts.txt which in turn means it is in upper-constratints.txt so you'll always get the version in upper-constraints.txt | |
| 14:34:26 | tonyb | sfinucan: do you have a review I can look at? but the bottom line is if you want to change *requirements.txt, right now, you need to do that in openstack/requirements.txt | |
| 14:35:01 | sfinucan | tonyb: Sure - one moments | |
| 14:37:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove literal block https://review.openstack.org/437923 | |
| 14:37:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Don't put comments inside toctree https://review.openstack.org/437924 | |
| 14:37:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Cleanup namespaces in nova-config-generator.conf https://review.openstack.org/437925 | |
| 14:37:06 | openstackgerrit | Stephen Finucane proposed openstack/nova master: setup.cfg: Add note to enable doc warnings https://review.openstack.org/437926 | |
| 14:37:20 | sfinucan | tonyb: ^ the last one of those is what I want to do | |
| 14:37:33 | tonyb | sfinucan: cool | |
| 14:38:01 | sfinucan | I thought I could because we install deps from test-requirements.txt (https://github.com/openstack/nova/blob/master/tox.ini#L18) | |
| 14:38:26 | sfinucan | ...which specifies a supported version of Sphinx (https://github.com/openstack/nova/blob/master/test-requirements.txt#L17) | |
| 14:39:02 | sfinucan | but ultimately it's the upper-constraints version that we use, which leads me to think that line in test-requirements is worthless | |
| 14:40:00 | tonyb | sfinucan: okay, that's strange | |
| 14:42:48 | tonyb | sfinucan: okay so basically somehow the requirements repo is messed up I'll go through the history to work out exactly how we got into that state | |
| 14:43:17 | tonyb | but the bottom line is that what you want to do should have been okay but $human messed up | |
| 14:43:29 | sfinucan | tonyb: So it wasn't just me :) OK, cool | |
| 14:44:00 | sfinucan | If you do fix it though, we might want to knock Sphinx down to 1.4 temporarily, because 1.5 is broken with the latest pbr https://review.openstack.org/#/c/430618/ | |
| 14:54:08 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: ironic inventory reporting https://review.openstack.org/437602 | |
| 14:55:05 | jaypipes | cdent: hey, I'm getting the following error on the functional test I just added in ^ patch: http://paste.openstack.org/show/600383/ | |
| 14:55:28 | jaypipes | cdent: I got this when I created this functional test on the old nova/ironic patch but can't remember what I did to fix... | |
| 14:55:35 | jaypipes | cdent: any insight would be appreciated. | |
| 14:56:08 | jaypipes | cdent: specifically, it's this test: https://review.openstack.org/#/c/437602/2/nova/tests/functional/compute/test_resource_tracker.py | |
| 14:56:17 | cdent | jaypipes: it ended up not being a problem with wsgi_intercept, I think it was a missing keystone related mock | |
| 14:56:30 | jaypipes | cdent: yeah, can't remember though... | |
| 14:56:38 | jaypipes | cdent: oh, shit... I know what it might be. | |
| 14:56:54 | jaypipes | I think we landed something yesterday that did a load_auth_from_conf() or something? | |
| 14:57:09 | gmann | http://logs.openstack.org/98/437898/1/experimental/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/fb81298/logs/tempest.txt.gz#_2017-02-24_14_00_57_338 | |
| 14:57:22 | cdent | jaypipes: yeah | |
| 15:00:41 | mordred | rcarrillocruz: oh wow. I'm super excited about that | |
| 15:00:57 | smcginnis | jaypipes: Any chance you might have a few minutes to give the Cinder team (and Manila representation) a quick overview of the long term plan for placement-api? | |
| 15:01:16 | jaypipes | smcginnis: for sure. | |
| 15:01:28 | jaypipes | smcginnis: just wrapping up a little bug right now... | |
| 15:01:36 | jaypipes | smcginnis: maybe in about 30 minutes? | |
| 15:01:38 | smcginnis | jaypipes: We're in ATL 1-2 if you want to stop by when you have time. | |
| 15:01:45 | smcginnis | jaypipes: Sounds great, thanks! | |
| 15:01:47 | jaypipes | smcginnis: cool. | |
| 15:02:15 | jaypipes | cdent: think I got it... | |
| 15:02:27 | jaypipes | cdent: all goes back to the scheduler_client vs. report_client thing.... | |
| 15:02:35 | cdent | nnnngh | |
| 15:02:42 | jaypipes | cdent: I'm setting RT.report_client instead of RT.scheduler_client | |
| 15:04:17 | jaypipes | cdent: yep, that was it. | |
| 15:08:27 | mriedem | sean-k-mooney: do you want to talk about accelerating sriov with dpdk? | |
| 15:09:26 | mriedem | ralonsoh: are you at the PTG? if so, show up to the nova room (georgia 9) to talk about your sriov stuff | |
| 15:09:42 | ralonsoh | I'm going now | |
| 15:11:03 | melwitt | dansmith: if the ClientRouter stuff isn't going to work, then I guess we'll need to revert https://review.openstack.org/#/c/326906 and then build upon the revert? | |
| 15:11:52 | dansmith | melwitt: I have a hack ready to push soon, but yeah, nearly a revert I think, but I will salvage some of what is there | |
| 15:12:06 | melwitt | okay | |
| 15:17:09 | mriedem | sfinucan: weren't you removing run_tests.sh from nova? | |
| 15:17:22 | sfinucan | mriedem: I did? | |
| 15:17:39 | mriedem | i thought you had a patch while you were cleaning up the unused venv and colorizer stuff | |
| 15:18:11 | mriedem | sfinucan: nevermind then, someone was asking me about removing it | |
| 15:21:01 | sfinucan | mriedem: Yeah, I'm pretty certain I did that and they've merged | |
| 15:21:40 | sfinucan | mriedem: Ah, no - I didn't do that one. I will https://review.openstack.org/#/q/status:merged+project:openstack/nova+branch:master+topic:feat/tools-cleanup | |
| 15:21:41 | bauzas | sfinucan: you did the change for having a specific tox target for osprofiler ? | |
| 15:21:52 | sfinucan | bauzas: https://review.openstack.org/#/c/437055/ | |
| 15:22:49 | bauzas | sfinucan: to be fair, we need a specific job in infra that would run that target as a Depends-On | |
| 15:24:00 | sfinucan | bauzas: Yeah, I wasn't sure if it would run automatically because I added it to the default targets or no | |
| 15:24:36 | mriedem | sfinucan: i just told this person to deal with run_tests.sh | |
| 15:24:39 | mriedem | sfinucan: so let them | |
| 15:24:57 | sfinucan | (y) | |
| 15:25:13 | bauzas | sfinucan: I doubt we run UTs in the gate by calling tox without args :p | |
| 15:25:18 | sdague | EmilienM: I left some debugging comments in the trippleo bug, I think at this point we need a keystone person to help figure out what's coming back | |
| 15:27:38 | openstackgerrit | Ryan Rossiter proposed openstack/nova master: Remove run_tests.sh https://review.openstack.org/437947 | |
| 15:30:15 | EmilienM | sdague: ok thanks | |
| 15:30:49 | EmilienM | mriedem, sdague: could we revisit https://review.openstack.org/#/c/428424/ please? | |
| 15:30:55 | sdague | EmilienM: there is also no user identity being logged in the logs for API calls, so it's hard to figure out what is happening | |
| 15:30:59 | EmilienM | I think tripleo jobs bring good feedback | |
| 15:32:54 | rcarrillocruz | johnthetubaguy: oh, that for linking, that makes sense | |
| 15:36:53 | sdague | EmilienM: I'm definitely -1 on per commit runs of that | |
| 15:40:58 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Reserve NUMA nodes with PCI devices attached https://review.openstack.org/364468 | |
| 15:42:45 | EmilienM | sdague: did you read the commit message, | |
| 15:42:47 | EmilienM | ? | |
| 15:42:53 | EmilienM | "The theory here is that Nova devs are going to dig through these," | |
| 15:43:04 | EmilienM | you're wrong, if you read the commit message, I said the opposite | |
| 15:43:19 | EmilienM | " The | |
| 15:43:21 | EmilienM | job would be non-voting for now, please don't ignore it and report | |