Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-01
20:11:56 melwitt yeah. I didn't consider the possibility of un-depcrecating it in the code only, and leaving the release notes alone. I had been thinking I should back out every mention of it
20:14:08 artom gryf, hey, so, I can't really talk now, but I'll be around later tonight, or tomorrow
20:14:56 openstackgerrit Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125
20:14:56 openstackgerrit Balazs Gibizer proposed openstack/nova master: Ignore forcing of evacuation for nested instance https://review.openstack.org/606111
20:15:49 gryf artom, tomorrow will be fine
20:19:16 mriedem melwitt: i didn't really have something specific in mind, because it's a weird issue
20:19:33 mriedem if nova-compute in stein is expecting stuff in the consoleauth service, it shouldn't be deprecated
20:19:55 mriedem but is that only true if the token isn't in the cell db?
20:20:04 mriedem or you're using cells v1 - which is itself deprecated
20:20:48 mriedem as an aside, it seems we should have something in grenade for consoles....create a console on a server on the old side and verify the console token is still valid on the new side...
20:21:07 mriedem i'm not sure how easy that is though
20:26:22 melwitt well, in rocky the console proxy by default will _not_ fall back to nova-consoleauth -- you have to use the [workarounds]enable_consoleauth to make it look there. the snag was in the fact that compute/api stores the token in both nova-consoleauth and the database, and that's an artifact that I didn't understand from the original patches from paulmurray
20:26:46 melwitt that's the non-cells v1 case. for cells v1, it will always need nova-consoleauth
20:38:44 sean-k-mooney melwitt: so the only case where you always need console auth is cells v1 which is deprecated and for non cells v1 you only need it if you enable a workaround which is off by default
20:39:43 melwitt sean-k-mooney: non cells v1 you need it in rocky even if you didn't enable the workaround because compute/api is still storing a token auth in nova-consoleauth regardless. the console proxy doesn't have to have nova-consoleauth
20:40:20 sean-k-mooney melwitt: oh ok i see thank i missed that bit
20:40:37 mriedem i'm about to get kicked out of this coffee shop
20:40:44 melwitt it's confusing
20:41:10 mriedem maybe write something up to the ML to help digest and sort it out?
20:41:13 mriedem so others are on the same page?
20:41:20 dansmith melwitt: would changing the default just be a better route?
20:41:25 mriedem and give operators a heads up before they get to rocky
20:41:30 dansmith we _want_ it deprecated and you shouldn't need it right?
20:42:06 dansmith or is there something we actually missed in the conversion to service-less?
20:42:16 dansmith if the latter, why not just fix and backport that?
20:42:25 melwitt dansmith: yeah, but to "fix" that, we'd have to remove the calls in compute/api that regardlessly store the token auth in both the consoleauth service _and_ the database. me missing that fact is why it got deprecated in the first place
20:43:05 melwitt the proxy code was optionally using nova-consoleauth, the compute/api code was unconditionally using nova-consoleauth
20:43:35 dansmith is it just not making an rpc call unless the workaround is on?
20:44:09 melwitt yeah, the proxy won't fall back on nova-consoleauth if the workaround option isn't enabled
20:44:18 dansmith what I mean is,
20:44:33 dansmith can we just not make the rpc calls from the compute api code unless the workaround is enabled?
20:44:49 dansmith will that fix the problem such that they don't have to run the service?
20:44:51 melwitt yeah, I think we could do that
20:45:11 dansmith that seems massively better to me than confusing everyone (including some of us here) with a backported un-deprecation dance
20:45:27 dansmith since that was the original goal, we just missed some of it (right?)
20:45:43 melwitt I think that should do it, obviously after this fiasco I will have to test it and make 100% sure it would work
20:46:02 melwitt before I claim that's all we need
20:46:06 dansmith yeah, that'd be good :)
20:46:30 dansmith did we deprecate and not also change devstack to not deploy?
20:46:37 dansmith deploy consoleauth I mean
20:46:43 melwitt yeah
20:46:46 dansmith I would think that would have caught this
20:46:48 dansmith ack
20:46:50 melwitt aye
20:47:14 melwitt yeah, it'd have to do a "if cells v1" before deploy/not deploy but I think that would be fine
20:47:56 melwitt and maybe I could rig up a DNM test for the workaround being enabled to test that in the gate too
20:48:44 dansmith we have cellsv1 conditions
20:52:35 sean-k-mooney melwitt: you could make the rpc conditional and then make a devstack change that depends on it that disables deploying the consoleauth service to test it
20:53:12 melwitt yeah, that's what we're talking about doing
20:54:05 sean-k-mooney cool and then the DNM test change is to reassert that if consoleauth is deploy the workaround still well works
20:56:15 melwitt yeah, that's what I was thinking
21:39:02 openstackgerrit Chris Dent proposed openstack/nova master: Clean up header encoding handling in compute API https://review.openstack.org/606967
21:44:13 openstackgerrit Merged openstack/nova stable/queens: Revert "Make host_aggregate_map dictionary case-insensitive" https://review.openstack.org/605266
21:44:19 openstackgerrit Merged openstack/nova stable/queens: Enforce case-sensitive hostnames in aggregate host add https://review.openstack.org/605267
21:59:15 openstackgerrit Merged openstack/nova stable/pike: Fix instance evacuation with PCI devices https://review.openstack.org/590059
22:04:16 openstackgerrit Merged openstack/nova stable/pike: Update nova network info when doing rebuild for evacuate operation https://review.openstack.org/590070
22:37:26 mriedem so has the great consoleauth undeprecafacation been sorted out?
22:39:20 melwitt mriedem: yeah, I'm working on an alternative approach dan suggested, to add the [workarounds] option condition to the compute/api calls, and see if that will make it possible not to run nova-consoleauth if backported to rocky. I can test it with a devstack change stacked on top. and I think I can use a DNM devstack change to verify that things also work with the [workarounds] option enabled
22:40:20 mriedem so default to workaround enabled and people can disable that when they have drained their existing tokens in the consoleauth service?
22:40:31 mriedem wonder if there is a nova-status check we can add for this...
22:41:17 melwitt I was thinking not to change the default, because only people with long TTL will potentially care about avoiding an instant reset of consoles
22:41:35 melwitt and the workaround is for those peeps
22:41:46 melwitt default TTL is 10 min
22:47:21 sean-k-mooney i think 3 hours of debuging the neutron sriov nic agent is enough for tonight
23:00:24 sean-k-mooney ... so i decided to restack and the thing i was debugging for the last 3 hours is now working and the only codecages i have done is add logging
23:48:52 openstackgerrit melanie witt proposed openstack/nova master: WIP Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068
23:49:41 mriedem melwitt: dansmith: thanks for hitting those pike backports in https://etherpad.openstack.org/p/nova-ocata-em
23:50:38 melwitt thanks for etherpadding them
23:50:52 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Cross-cell resize https://review.openstack.org/603930
#openstack-nova - 2018-10-02
00:04:33 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle missing marker during online data migration https://review.openstack.org/605164
00:05:01 melwitt takashin: hi, could you please check the reply to your -1 on this change? https://review.openstack.org/582147
00:05:38 takashin melwitt: I will check it.
00:05:45 melwitt thank you
00:07:12 takashin melwitt: I would like to know other cores' opinions about https://review.openstack.org/#/c/582147/ .
00:07:46 melwitt takashin: ack. will try to look later
00:14:18 openstackgerrit Merged openstack/nova stable/pike: unquiesce instance after quiesce failure https://review.openstack.org/581454
00:53:38 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix test_instance_action functional test failure https://review.openstack.org/606947
01:34:25 openstackgerrit Brin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type`` https://review.openstack.org/579360
02:29:39 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088
02:32:26 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088
02:33:42 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088
02:51:10 openstackgerrit Merged openstack/nova stable/pike: Update RequestSpec.flavor on resize_revert https://review.openstack.org/605879
03:12:45 openstackgerrit Ghanshyam Mann proposed openstack/nova master: [WIP]Remove extensions loading framework from wsgi.py https://review.openstack.org/607092
03:21:54 openstackgerrit Merged openstack/nova stable/rocky: nova-manage - fix online_data_migrations counts https://review.openstack.org/605828
03:22:01 openstackgerrit Merged openstack/nova stable/pike: Filter deleted computes from get_all_by_uuids() https://review.openstack.org/604449
03:24:16 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge image_size extension response into image view builder https://review.openstack.org/606845
03:49:07 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix test_instance_action functional test failure https://review.openstack.org/606947
04:08:01 openstackgerrit Brin Zhang proposed openstack/nova master: Add compute API version for when a ``volume_type`` is requested https://review.openstack.org/605573
04:24:54 openstackgerrit Merged openstack/nova stable/pike: Use ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa in tree https://review.openstack.org/581444
06:22:03 openstackgerrit OpenStack Proposal Bot proposed openstack/nova stable/rocky: Imported Translations from Zanata https://review.openstack.org/604260
07:11:42 openstackgerrit Radoslav Gerganov proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116
07:54:40 bauzas good morning nova
08:05:01 gryf morning guys
08:11:03 bhagyashris Hi all, I am facing one issue on current master while live migration of the instance , I am trying to live migrate the instance but it's not migrating that instance on given destination node and that instance remains on source node with the status as active, checked n-cpu, n-condunctor logs but didn't get any error at first time.
08:12:06 bhagyashris And if I again try to live migrate that instance then it's giveing me Migration pre-check error: Binding failed for port <port-id> please check neutron logs for more information. (HTTP 400
08:12:58 bhagyashris Can anyone please help me
08:39:44 takashin Nova cores, could you review https://review.openstack.org/#/c/606947/ ? It is a fix for a gate failure in python-novaclient.
08:48:40 gibi bhagyashris: what do you see in the neutron log about the bind failure?
08:52:52 gibi bhagyashris: have you specified target host for the migration ? Have you use force=True in the migration request?
08:54:42 bhagyashris gibi: this is log I get at neutron server side http://paste.openstack.org/show/731235/

Earlier   Later