Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-20
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
06:29:11 openstackgerrit Chen proposed openstack/nova master: Specify default update_resources_interval https://review.openstack.org/576864
06:51:49 openstackgerrit hyunsik Yang proposed openstack/nova-specs master: Specification for support pxe/IPMI boot for instance https://review.openstack.org/577048
07:54:50 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object https://review.openstack.org/573628
07:54:51 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models. https://review.openstack.org/563375
07:54:52 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401
07:54:53 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166
07:54:54 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465
07:54:55 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534
07:56:47 belmoreira I'm trying to add a custom_resource class (GPU) to a resource_provider but the resource tracker overwrites the inventory and removes it. I'm missing something?
08:22:37 tssurya good morning gibi! there is an update on the handling the down cell spec (https://review.openstack.org/#/c/557369/) so dansmith and mordred gave their views last evening , they were suggesting its better to just bump microversion saying there may be missing values for keys, than introducing the new top level key ; dansmith has put this as a comment on the spec, me personally I don't mind going wither ways, let me know your view whenever
08:23:43 tssurya either*
08:49:54 openstackgerrit Vladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921
08:51:39 openstackgerrit Gergely Csatari proposed openstack/nova master: Adding oslo.service to test-requirements https://review.openstack.org/577095
09:27:07 openstackgerrit Chris Dent proposed openstack/nova master: Cleanup nits in placement database changes https://review.openstack.org/577107
09:31:01 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case https://review.openstack.org/574240
09:41:14 openstackgerrit Vishakha Agarwal proposed openstack/nova master: No requirement of –all-tenants while listing servers https://review.openstack.org/577111
09:45:45 gibi tssurya: ack. I have to read back then I will go and read the review
09:49:31 tssurya gibi: thanks so much for your time
10:10:39 openstackgerrit Vishakha Agarwal proposed openstack/nova master: No requirement of –all-tenants while listing servers https://review.openstack.org/577111

Earlier   Later