| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-15 | |||
| 14:44:52 | bauzas | fried_rice: is there a feature parity between ironicclient and openstacksdk ? | |
| 14:46:02 | bauzas | at least for compute, I can't see it https://docs.openstack.org/openstacksdk/latest/user/guides/compute.html | |
| 14:46:03 | fried_rice | bauzas: For many if not all of the APIs - creepy_owlet could answer that - but even if there's a specific API missing, we can use the adapter.get/put/post/delete primitives | |
| 14:46:55 | fried_rice | ...which is what I was starting to do (via ksa directly as opposed to sdk) via https://review.openstack.org/642899 which started this discussion. | |
| 14:48:13 | bauzas | I'm confused, I thought we were talking about the potential TC goal for Train | |
| 14:49:02 | bauzas | my sole take on using other clients but the project ones is that we need to make sure those projects take ownerships of their respective parts | |
| 14:49:54 | fried_rice | bauzas: Right, we're specifically *not* talking about the current community goal. We're talking about a potential future community goal that's similar, but different. Namely: stop using python-*client from *services* (as opposed to CLIs). | |
| 14:50:49 | mordred | fried_rice: https://review.openstack.org/643601 WIP Make factory for a CloudRegion from CONF objects | |
| 14:51:27 | fried_rice | mordred: ack | |
| 14:51:30 | mordred | fried_rice: that's probably 95% there - obviously needs tests and whatnot - but that's the overall idea | |
| 14:52:22 | mordred | fried_rice: I think - because nova supports people having different accounts for different services, for now we should keep the existing model and have your nova code load session and auth plugin and pass them in - and we'll do one-connection-per-service - even though that's ultimately a little strange | |
| 14:52:44 | fried_rice | mordred: okay, I was wondering about that. | |
| 14:52:46 | mordred | fried_rice: but fits with the consumption model better so we can keep things as it is | |
| 14:53:06 | mordred | then, later, we can talk about whether it makes sense to think about it differently | |
| 14:54:49 | mordred | fried_rice: anyway - I've got a call now, so I'll have to come back to it -but I don't think it'll be super hard to get a working poc based on that | |
| 14:55:02 | fried_rice | ack | |
| 14:56:31 | creepy_owlet | bauzas: there is no feature parity overall, but most of things needed for Nova are there. | |
| 14:57:04 | creepy_owlet | One missing is Volume API, but it should be easy to introduce now that we have all basic bits in place. | |
| 14:57:34 | mordred | and it woudl be good all around to get that added to sdk anyway, so the work would be generally useful | |
| 14:58:15 | creepy_owlet | exactly | |
| 14:58:43 | creepy_owlet | FYI I'd like to start Train with replacing ironicclient with openstacksdk in ironic-inspector | |
| 14:58:54 | creepy_owlet | metalsmith is already 100% openstacksdk | |
| 14:59:28 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove [ironic]api_endpoint option https://review.openstack.org/643483 | |
| 15:16:46 | melwitt | aspiers: heya, was wondering if you're up for writing up a doc for https://review.openstack.org/538498 for rc1? L38 https://etherpad.openstack.org/p/nova-stein-rc-potential | |
| 15:19:48 | mriedem | fried_rice: i was out, but i have an answer, sec | |
| 15:20:39 | mriedem | fried_rice: i mentioned historical context in the community wide goal about sdk usage in server projects and dtroyer replied https://review.openstack.org/#/c/639376/4/goals/train/osc.rst@33 | |
| 15:20:52 | mriedem | tl;dr is it used to be heavyweight to use the sdk | |
| 15:20:59 | mriedem | now it's not so much so we should be ok to use if we want | |
| 15:26:28 | mriedem | but i see dmitry and monty already told you all this | |
| 15:40:51 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove stale aggregates notes from scheduler evolution doc https://review.openstack.org/643613 | |
| 15:40:54 | mriedem | bauzas: edleafe: ^ | |
| 15:44:55 | edleafe | mriedem: so is the plan for the evolution doc to remove piece-by-piece instead of all at once? | |
| 15:44:58 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove "Fixing the Scheduler DB model" from schedule evolution doc https://review.openstack.org/643615 | |
| 15:45:08 | mriedem | edleafe: some of it needs re-wording, | |
| 15:45:12 | mriedem | it's not all removal | |
| 15:45:23 | edleafe | mriedem: ah, ^^ answers that | |
| 15:45:30 | mriedem | and i'd rather not try to bite off re-writing that entire thing in a single change because (1) i don't have the time and (2) people probably don't want to review it | |
| 15:45:50 | mriedem | so i'm just killing some time before lunch by hitting the low hanging fruit | |
| 15:49:54 | openstackgerrit | Merged openstack/nova stable/queens: Provide a useful error message when trying to update non-compute services https://review.openstack.org/638803 | |
| 15:51:20 | mriedem | melwitt: want to hit this for rocky https://review.openstack.org/#/c/643173/ | |
| 15:51:45 | openstackgerrit | Merged openstack/nova stable/queens: Fix incompatible version handling in BuildRequest https://review.openstack.org/634439 | |
| 15:51:51 | melwitt | mriedem: sure | |
| 15:52:21 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Fix resetting non-persistent fields when saving obj https://review.openstack.org/643215 | |
| 15:53:35 | mriedem | oh right dan is out today and monday, forgot about that | |
| 15:55:00 | mriedem | https://review.openstack.org/#/c/643215/ is also needed for rocky | |
| 17:33:41 | ravirjn | Hi everyone! We have one running newton trippleo cluster. Just now we added HP MSA storage. Now, we want to migrate all nova instances from compute host to cinder volume. But, we are not sure how to migrate instances to cinder volume. For testing, we created a snapshot of a running instance, and created new instance using this instance-snapshot, but, the problem is, the IP addresses are getting changed. Please help. | |
| 17:47:29 | openstackgerrit | Merged openstack/nova stable/rocky: Check hosts have no instances for AZ rename https://review.openstack.org/641351 | |
| 17:47:37 | openstackgerrit | Merged openstack/nova master: docs: cleanup driver parity scope section https://review.openstack.org/643566 | |
| 17:47:46 | openstackgerrit | Merged openstack/nova master: Remove resize caveat from conductor docs https://review.openstack.org/643567 | |
| 17:47:55 | openstackgerrit | Merged openstack/nova master: Add descriptions of numbered resource classes and traits https://review.openstack.org/621494 | |
| 17:48:03 | openstackgerrit | Merged openstack/nova master: Trivial typo fix for REST API in policy enforcement docs https://review.openstack.org/643569 | |
| 17:51:50 | mdbooth | melwitt: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003870.html | |
| 17:52:01 | mdbooth | The monkey patching thing | |
| 17:53:14 | mdbooth | Incidentally, I clearly don't understand how we bump requirements. | |
| 17:53:50 | mdbooth | I wanted to bump the eventlet version requirement to >=0.21.0. That seems to have failed because we're using 0.18.2, but that seems super old. | |
| 17:54:32 | mdbooth | Hmm, that's the lower constraint | |
| 17:55:13 | sean-k-mooney | mdbooth would bumping the min version be blocked as we are after the non client lib freeze | |
| 17:55:27 | mdbooth | Ah, presumably we only install the old stuff for the lower constraints test | |
| 17:56:12 | sean-k-mooney | mdbooth: in the gate yes. that basically just runs the unit tests | |
| 17:56:18 | mdbooth | sean-k-mooney: No idea. It's not critical, tbh, as I can easily take that out. I was more interested in why we haven't hit this in the gate yet. | |
| 17:56:44 | mdbooth | We have a workaround in the monkey patching which is no longer required after eventlet 0.21.0 | |
| 17:56:51 | sean-k-mooney | mdbooth: ya as i said downstream from looking at what we run in the gate we should | |
| 17:57:01 | mdbooth | So while I was in there with a test patch I bumped the eventlet min version and removed it. | |
| 17:57:15 | mdbooth | I assumed it would go through as we'd be using a newer version already anyway. | |
| 17:58:14 | sean-k-mooney | we should not be useing the lower constraint by the way in the tempest job | |
| 17:58:40 | sean-k-mooney | mdbooth: perhaps this is showing up in osp15 because we are packagin an old eventlet | |
| 17:58:53 | mdbooth | sean-k-mooney: Nope. Latest and greatest. | |
| 17:59:24 | sean-k-mooney | the tempest-full-py3 job will use whatever the uppper-constraits.txt allows | |
| 17:59:52 | melwitt | 0.24.1 is the upper constraint for eventlet, so I'm not sure why 0.18.2 is being installed https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L143 | |
| 17:59:54 | sean-k-mooney | we should be running 0.20.0 | |
| 17:59:55 | sean-k-mooney | https://github.com/openstack/requirements/blob/stable/pike/upper-constraints.txt#L141 | |
| 18:00:01 | sean-k-mooney | oh that was pike | |
| 18:00:14 | mdbooth | 0.24.1 | |
| 18:00:18 | mdbooth | Same as OSP | |
| 18:00:23 | mdbooth | Might be urllib3, then | |
| 18:00:29 | sean-k-mooney | eventlet===0.24.1 on master | |
| 18:00:46 | sean-k-mooney | urllib3===1.24.1 | |
| 18:00:49 | sean-k-mooney | on master | |
| 18:01:04 | mdbooth | OSP: python3-urllib3-1.23-5 | |
| 18:01:11 | mdbooth | Slightly older urllib3 | |
| 18:01:44 | mdbooth | Anyway, as I said in the ML post it's not that important. By not monkey patching first, if it's not this today it will be something else tomorrow. | |
| 18:03:03 | sean-k-mooney | ya | |
| 18:03:41 | sean-k-mooney | at somepoint maybe we can finally move away form eventlets entriely but it will atleast be U before that becomes possible | |
| 18:04:45 | sean-k-mooney | i.e. we need to drop python2 support first | |
| 18:20:31 | openstack | Launchpad bug 1801303 in OpenStack Compute (nova) "VM send DHCP request before openflow is created in br-int" [Undecided,New] | |
| 18:20:31 | melwitt | sean-k-mooney: you might be able to help triage this bug, looks in your wheelhouse https://bugs.launchpad.net/nova/+bug/1801303 | |
| 18:26:02 | openstackgerrit | Merged openstack/nova stable/rocky: Update instance.availability_zone during live migration https://review.openstack.org/643173 | |
| 18:32:59 | sean-k-mooney | melwitt: sure ill take a look now | |
| 18:33:31 | melwitt | sean-k-mooney: thanks | |
| 18:34:06 | sean-k-mooney | my inital reaction is this shound never happen in any release after we added the vif plugged events as we wait for those on boot | |
| 18:34:58 | melwitt | oh, I see. I think it's been awhile since those were added | |
| 18:35:10 | melwitt | i.e. before queens | |
| 18:35:19 | sean-k-mooney | yes back in like mitaka or so queens should have it | |
| 18:35:32 | sean-k-mooney | im going to ask from mor info on what network backend they are using | |
| 18:35:44 | melwitt | ok, cool | |
| 18:35:45 | sean-k-mooney | if they are using odl then that would explain the behavior | |
| 18:36:02 | melwitt | oh, right. I vaguely remember that from last time I tried to think about vif plug events xD | |
| 18:36:04 | sean-k-mooney | odl sends the event on port bind not when odl wires it up | |
| 18:50:46 | fried_rice | mordred: ! | |
| 18:52:03 | fried_rice | mordred: I pulled down your openstacksdk patch, diddled with it a little bit, added a kwarg to Connection, instantiated it with a CONF and a session/auth established from ks_loading, and succeeded in talking to placement through it. | |