Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-25
14:13:50 efried stephenfin: You had a -1 on a previous PS of https://review.opendev.org/#/c/683783/ -- I think your feedback was addressed (repath was reverted) but would you please have another quick look and make sure there's nothing blocking remaining?
14:14:55 bauzas gibi: mriedem: honestly, not a lot of people work for 100% upstream you know
14:15:08 bauzas so we have a lot of other stuff
14:15:09 efried sean-k-mooney: I'll proxy your approval of https://review.opendev.org/#/c/683437/ if you could have another look please ("everything CPU pinning" doc)
14:15:54 bauzas efried: like I said previously, unless a doc is having some wrong explanation, I should we should just accept it, and then providing some FUP
14:16:00 bauzas (in case of comments)
14:16:01 gibi bauzas: I would be happy to get an extra two half-people for stable/pike from my employee but...
14:16:56 efried bauzas: I agree; I'm trying to delegate to someone else to confirm that there's nothing actually wrong in there.
14:18:03 bauzas stephenfin: FWIW, finally looking at https://review.opendev.org/#/c/667133 :)
14:21:54 mriedem bauzas: i thought most people were 100% upstream...
14:22:15 bauzas yeah, me too honestly
14:22:32 mriedem sorry, i'm being facetious
14:22:40 bauzas and then I discovered the other way once I went a core
14:22:52 bauzas because 'heh, can you do this please'
14:22:59 bauzas or 'look at this, is this right?'
14:36:06 mriedem gibi: looks like the show version tempest test failures in https://review.opendev.org/#/c/684658/ are real (on pike)
14:36:24 mriedem tempest doesn't test pike since it's in extended maintenance mode so it's likely something changed in tempest
14:36:47 mriedem probably dropped support for cinder v1 or v2 or something
14:39:44 mriedem this https://review.opendev.org/#/c/680790/
14:39:51 mriedem gmann: stable/pike is also broken since ^
14:40:57 mriedem so i guess we're going to have to run with a tagged tempest on stable/pike as well
14:41:17 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Stop silently ignoring invalid 'nova boot --hint' options https://review.opendev.org/684762
14:41:31 stephenfin that novaclient unit tests are dumb and I hate them
14:42:33 gibi mriedem: thanks
14:42:41 gibi mriedem: I will have to get back to that
14:43:24 mriedem my guess is tempest is failing b/c the cinder v1 api was removed in queens https://review.opendev.org/#/c/499342/
14:46:35 gibi I try to get elod look into this
14:51:24 mriedem gibi: just posted to the ML
14:54:02 gibi mriedem: ack, thanks
14:58:09 mriedem https://review.opendev.org/#/c/684769/
15:03:16 openstackgerrit Martin Midolesov proposed openstack/nova master: Implementing graceful shutdown. https://review.opendev.org/666245
15:06:14 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Ignore warning from sqlalchemy-migrate https://review.opendev.org/684772
15:06:15 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774
15:06:15 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773
15:06:16 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775
15:06:30 openstackgerrit Balazs Gibizer proposed openstack/nova stable/pike: Delete instance_id_mappings record in instance_destroy https://review.opendev.org/684658
15:06:32 stephenfin mriedem: You'd asked for the first of those last week ^
15:07:19 mnaser mriedem: ouch, i think it should be easy to fix .. i hope :X
15:10:17 mriedem stephenfin: so about that,
15:10:30 mriedem i was trying to do that last week myself and found the warnings go away for py27 but not py36 runs
15:12:17 mriedem stephenfin: comments in there, but the py removal is wrong, and we should split fixing that out since it's on stable branches
15:13:56 stephenfin mriedem: Oh, so it is (called py). My eyes completely glazed over that \o/
15:14:09 stephenfin fix coming right up
15:14:40 mriedem stephenfin: also a question in the nuke patch after that
15:15:27 openstack bug 1813147 in OpenStack Compute (nova) "p35 jobs are failing with subunit.parser ... FAILED" [High,In progress] https://launchpad.net/bugs/1813147 - Assigned to Balazs Gibizer (balazs-gibizer)
15:15:27 mriedem track that warning ignore patch against bug 1813147 as well
15:15:28 mriedem please
15:18:54 mriedem dansmith: care to get https://review.opendev.org/#/c/684140/ so the cross_az_attach func test patch can be unblocked?
15:19:20 dansmith oh sorry did I miss that yesterday?
15:21:55 gregwork is there an equivalent of nova get-password in the openstack client
15:22:55 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774
15:22:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Ignore warning from sqlalchemy-migrate https://review.opendev.org/684772
15:22:56 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Nuke OS_STDOUT_CAPTURE, OS_STDERR_CAPTURE https://review.opendev.org/684773
15:22:56 openstackgerrit Stephen Finucane proposed openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775
15:22:57 openstackgerrit Stephen Finucane proposed openstack/nova master: Ignore sqla-migrate inspect.getargspec deprecation warnings on py36 https://review.opendev.org/684781
15:25:34 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Handle legacy request spec dict in ComputeTaskManager._cold_migrate https://review.opendev.org/684407
15:27:34 mriedem gregwork: nope
15:27:52 mriedem could throw it at the bottom of https://etherpad.openstack.org/p/compute-api-microversion-gap-in-osc
15:28:30 mriedem (i just did)
15:28:54 gregwork i saw saw that :)
15:28:56 gregwork *just
15:29:46 gregwork getting our windows team members to become familiar with the openstack client and one of the things they need for new instances is to nova get-password server rsa-key .. trying to streamline the tools they start off with
15:30:24 mriedem yeah definitely want new people using osc if they can rather than the mish-mash of per-project CLIs
15:30:32 mriedem but someone would have to add that to osc
15:30:48 mriedem it would be a pretty easy addition to osc
15:40:28 mriedem gregwork: the question is if you have developers?
15:41:37 gregwork so its early days for my org, it would probably be me writing something like that. We are in the middle of a prod build of openstack where the broader organization must now get involved.
15:42:12 gregwork it would also need to be something id get our vendor to backport
15:42:19 mriedem ok, shouldn't really be too hard, it's mostly just argparse stuff in osc and it calls through to the novaclient python api bindings
15:42:30 gregwork we are definitely not brave enough to run upstream in production
15:42:34 mriedem backport how far? queens?
15:42:42 mriedem ah red hat
15:42:51 gregwork wha .. who where ?
15:42:56 gregwork :D
15:43:00 mriedem if only we (nova) had some red hat developers around
15:44:24 gregwork indeed
15:49:13 mriedem mlavalle: routed networks is not supported in nova yet, no - see https://review.opendev.org/656885
15:49:46 mriedem i don't plan on actively pushing that forward
15:50:50 mlavalle mriedem: if my employer needs, this, could we push it forward in U?
15:52:40 mriedem you'll have to talk to the boss, efried
15:53:19 mlavalle mriedem: thanks for following up :-)
15:53:24 mriedem yw
15:53:31 efried mlavalle: Depends who your employer is
15:53:35 efried I kid, I kid.
15:53:53 mlavalle would you be open to use pushing this forward?
15:53:59 efried what are we talking about?
15:54:01 mlavalle to us^^^
15:54:36 kashyap efried: https://review.opendev.org/#/c/656885/
15:54:58 kashyap Ah, you were jesting
15:55:21 efried Has there been a bp and/or spec at any point?
15:55:52 efried is this related? https://review.opendev.org/#/c/263898/
15:56:19 mlavalle this is the approved spec: http://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/neutron-routed-networks.html?
15:56:46 mlavalle yeah same thing....
15:57:44 efried mlavalle: I'm toying with ideas for ways to dramatically constrain the scope of what we commit to in ussuri, on the theory that stakeholders (such as your employer, whoever that may be) would rather get a hard "no" up front than have a bp/spec approved and have NO IDEA whether it's going to land, based on our track record of landing a random 60-ish percent of the things we approve, with no relation to the supposed prioritization
15:58:15 mriedem mlavalle works for verizon media with jroll and penick and the gang
15:58:46 mlavalle LOL, I didn't know we are the gang
15:58:47 mriedem the routed networks stuff in that spec back in newton was a lot more grandiose with refactoring all of the port creation stuff in conductor - a lot of stuff johnthetubaguy worked on while at OSIC
15:59:09 mriedem what functionally landed on the nova side was actually a really small change to allow deferred fix ip allocation
15:59:27 mriedem for which https://review.opendev.org/#/c/641670/ is a follow up
15:59:44 mriedem the actual routed networks thing in nova is a lot more complicated,

Earlier   Later