Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-20
17:16:37 dansmith tssurya: yeah
17:16:42 tssurya infact just the uuid would also be good enough with the state as UNKNOWN like you said
17:16:58 tssurya and we would anyways need a "odd microversion" bump
17:17:23 mordred hrm. well - it would make things a bit awkward for ansible dynamic inventory without name ... but if that's life, I guess that's life
17:17:56 tssurya mordred: :)
17:18:19 mordred basically it'll make operations that are working with servers that are up not able to find the server if part of the api control is down if/when those servers are being managed by name and not uuid
17:18:22 mordred BUT
17:18:53 mordred maybe that's just life - the existing system would mean you can't deal with ANY of your servers - so it's still an improvement
17:19:08 dansmith yeah
17:19:15 dansmith if we do the 1a option then it gets better
17:19:18 mordred yup
17:19:28 tssurya yea
17:19:40 mordred so yeah - I think 1 followed by 1a would be great
17:20:23 dansmith mordred: thanks for the discussion!
17:20:26 tssurya mordred: totally appreciate your view and opinion on this :) thanks!
17:20:49 tssurya dansmith: so you will write a comment on yours and mordred's behalf in the spec ?
17:20:58 dansmith tssurya: am now yeah
17:21:04 tssurya and then we can discuss it with mriedem and gibi next week
17:21:18 tssurya dansmith: thanks!
17:21:29 dansmith yup
17:29:10 openstackgerrit Merged openstack/nova master: Optimize member_of check for nested providers https://review.openstack.org/573203
17:45:55 zcorneli stephenfin: Ack. I'll fix that up.
18:00:39 dansmith zcorneli: bonus points if you fix up the stuff in the devstack patch so we can merge that
18:00:45 dansmith if not, I'll try to get around to it at some point
18:01:13 zcorneli Sure, IIRC it looked pretty simple/quick to fix up the stuff mriedem was looking for.
18:01:27 dansmith yup
18:01:28 zcorneli dansmith: I've also got the discard patch almost ready to review as well.
18:01:34 dansmith zcorneli: excellent
18:02:05 zcorneli dansmith: Going to finish the fix up patch, rebase onto it, then post, I think.
18:02:14 zcorneli Then I can take a crack at the devstack fixup.
18:02:19 dansmith cool
18:11:09 openstackgerrit Chris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST https://review.openstack.org/576927
18:23:02 openstackgerrit Merged openstack/nova master: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857
18:39:45 mnaser so mriedem was working on a patch adding `nova-status upgrade check as part of openstack ansible deployment which is really sweet, but one of the issues we're having is detecting what action we're missing
18:39:56 mnaser is it possible to perhaps have a different exit code depending on which 'issue'
18:40:13 mnaser in this case, upgrading from queens to master (rocky), we need to restart placement
18:41:15 mnaser like maybe some sort of .. cli friendly output so we can run `nova-status upgrade check` once
18:41:35 mnaser then have conditionals depending if it returns something like out_of_date_placement => restart placement, etc
18:42:21 dansmith returning different statuses per check would make it unstable across releases, which is probably undesirable
18:42:38 dansmith how about just some sort of robot-friendly output?
18:42:57 mnaser yeah i was just thinking how complicated that would become at some point
18:43:10 mnaser but robot friendly output, something like one line per 'check' .. only if there is something
18:43:25 mnaser that way, we can run it once, ignore exit code, and then have tasks after it conditional if output matches that string
18:43:26 dansmith like --only-critical
18:43:36 dansmith and one per line, csv-separated or something
18:43:41 mnaser yes
18:43:48 dansmith --only-critical --robot-friendly
18:44:10 cdent --with-ponies
18:44:17 mnaser --emojis
18:44:18 cdent --and-pudding
18:44:23 cdent exactly
18:44:26 dansmith too far.
18:45:05 mnaser there is 6 warnings and 11 failures
18:45:15 mnaser UpgradeCheckResult
18:45:17 mnaser https://github.com/openstack/nova/blob/master/nova/cmd/status.py
18:45:17 mnaser er
18:45:34 mnaser UpgradeCheckResult has 'user friendly' messages but we can just add another value to it
18:46:40 mnaser i think ill push up a patch for that
19:05:52 openstackgerrit Lee Yarwood proposed openstack/nova master: Add regression test for bug #1764883 https://review.openstack.org/562072
19:05:53 openstack bug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,In progress] https://launchpad.net/bugs/1764883 - Assigned to Lee Yarwood (lyarwood)
19:05:54 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Ensure pre-migrating instances are destroyed during init_host https://review.openstack.org/562284
19:21:47 openstackgerrit Mohammed Naser proposed openstack/nova master: Add machine output for nova-status upgrade checks https://review.openstack.org/576944
19:21:49 mnaser dansmith: ^ my first attempt.. not sure who might be best to look over it too :)
19:21:52 mnaser we can use that in openstack ansible
19:26:40 mordred mnaser: are there any projects you aren't hacking on?
19:27:37 mnaser mordred: :D i'm sure there is a quite a few
19:31:54 dansmith mnaser: mriedem is the right person :)
19:33:12 mnaser dansmith: cool, thanks
19:40:32 openstackgerrit Carlos Camacho proposed openstack/nova stable/queens: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576948
20:41:43 openstackgerrit Merged openstack/nova master: Fix nits from change Id609789ef6b4a4c745550cde80dd49cabe03869a https://review.openstack.org/574324
20:52:01 openstackgerrit Matthew Edmonds proposed openstack/nova master: fix PowerVM get_bootdisk_path docstring https://review.openstack.org/576977
20:52:11 edmondsw efried esberglu ^
20:53:57 esberglu edmondsw: LGTM tx
20:56:49 artom afazekas, gonna head home for the day, I've put an update in the bz. You've handed me a puzzler :/
20:56:57 artom Grr, was meant for our downstream chan
20:58:00 openstackgerrit Merged openstack/nova master: Implement file backed memory for instances in libvirt https://review.openstack.org/567876
21:10:07 openstackgerrit Eric Fried proposed openstack/nova master: Tighten up ReportClient use of generation https://review.openstack.org/556669
21:13:09 openstackgerrit Zack Cornelius proposed openstack/nova master: Fix nits from change I676291ec0faa1dea0bd5050ef8e3426d171de4c6 https://review.openstack.org/576985
21:13:42 zcorneli dansmith: stephenfin: ^
21:14:11 openstackgerrit Eric Fried proposed openstack/nova master: Tighten up ReportClient use of generation https://review.openstack.org/556669
21:53:21 openstackgerrit Zack Cornelius proposed openstack/nova master: Implement discard for file backed memory https://review.openstack.org/576994
23:52:05 openstackgerrit Merged openstack/nova master: Comment proposed ironic fix for removal of ironic driver workaround https://review.openstack.org/576892
#openstack-nova - 2018-06-21
01:02:57 openstackgerrit huanhongda proposed openstack/nova stable/ocata: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/570653
01:30:45 yikun gibi, thanks for you review, I will address them soon.
02:21:12 openstackgerrit Chen proposed openstack/nova master: Specify default update_resources_interval https://review.openstack.org/576864
02:24:50 openstackgerrit Dinesh Bhor proposed openstack/nova master: PCPU: Define numa dedicated CPU resource class https://review.openstack.org/561770
02:24:51 openstackgerrit Dinesh Bhor proposed openstack/nova master: PCPU: Add respective conf options https://review.openstack.org/561771
02:24:52 openstackgerrit Dinesh Bhor proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes https://review.openstack.org/576021
02:27:53 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add instance.lock notification https://review.openstack.org/576702
02:30:17 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add instance.lock notification https://review.openstack.org/576702
02:41:19 openstackgerrit Chason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023
03:01:04 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification https://review.openstack.org/576703
03:12:25 openstackgerrit Tony Breeds proposed openstack/os-vif master: Fix docstrings to work with Sphinx 1.7.4 https://review.openstack.org/577028
04:17:53 openstackgerrit Dinesh Bhor proposed openstack/nova master: Report PCPU to placement https://review.openstack.org/577038
04:23:15 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487
04:28:22 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series https://review.openstack.org/573604
05:08:45 openstackgerrit hyunsik Yang proposed openstack/nova-specs master: Specification for support pxe/IPMI boot for instance https://review.openstack.org/577048
06:15:25 openstackgerrit hyunsik Yang proposed openstack/nova-specs master: Specification for support pxe/IPMI boot for instance https://review.openstack.org/577048
06:17:01 openstackgerrit Chen proposed openstack/osc-placement master: Fix docstring for delete allocation method https://review.openstack.org/577075
06:24:35 openstackgerrit garyk proposed openstack/nova master: VMware: populate datastore refs at init https://review.openstack.org/574688

Earlier   Later