Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-29
02:37:32 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
02:37:33 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
02:37:34 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
02:37:35 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
02:37:36 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
02:37:37 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
02:39:17 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Remove doc/build during tox -e docs https://review.openstack.org/579041
02:41:27 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Fix the 1.6 release note format https://review.openstack.org/579042
02:44:43 openstackgerrit Merged openstack/nova master: Skip ServerShowV247Test.test_update_rebuild_list_server in nova-cells-v1 job https://review.openstack.org/578125
05:06:25 openstackgerrit Andreas Jaeger proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
05:43:46 openstackgerrit Merged openstack/nova master: VMware: remove reading resourcePool data https://review.openstack.org/574553
05:43:55 openstackgerrit Merged openstack/nova master: sync_guest_time: use the proper errno https://review.openstack.org/572346
05:44:04 openstackgerrit Merged openstack/nova master: Downgrade overquota warning https://review.openstack.org/572028
05:44:28 openstackgerrit Merged openstack/nova master: Use nova.test.TestingException https://review.openstack.org/575012
05:44:51 openstackgerrit Merged openstack/nova master: update the description of hypervisor statistics response https://review.openstack.org/462355
06:28:13 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for scheduler hint extension https://review.openstack.org/579067
06:32:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server create for scheduler hint extension https://review.openstack.org/579067
06:58:54 openstackgerrit Zhenyu Zheng proposed openstack/nova master: API: add support to abort queued live migration in microversion 2.64 https://review.openstack.org/573136
06:59:29 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
07:12:36 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/578019
07:18:50 openstackgerrit Merged openstack/nova master: conf: libvirt: Make `/dev/urandom` the default for 'rng_dev_path' https://review.openstack.org/577385
07:18:57 openstackgerrit Merged openstack/nova master: Update scheduler to use image-traits https://review.openstack.org/576054
07:23:07 openstackgerrit Merged openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
07:23:14 openstackgerrit Merged openstack/nova master: Powervm configuration cleanup https://review.openstack.org/575171
07:23:24 openstackgerrit Merged openstack/nova master: placement: s/None/null/ in consumer conflict msg https://review.openstack.org/577171
08:02:45 lyarwood gibi: https://review.openstack.org/#/c/483324/10/nova/compute/manager.py@6042 - just noticed some additional whitespace in an old change of yours, odd how pep8 didn't pick up on it.
08:27:24 openstackgerrit huanhongda proposed openstack/nova stable/queens: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/579093
08:33:40 gibi lyarwood: yeah, that is odd
08:38:01 bauzas shit, my manners
08:38:04 giblet lyarwood: do you feel it worth a patch to remove tht extra line?
08:42:49 lyaaaaaaaarwood giblet: honestly no, was just reading the LM rollback code and noticed it.
08:44:32 giblet lyaaaaaaaarwood: OK, let's keep it that way. I have no idea if we can teach pep8 to detect this
08:51:13 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487
08:51:14 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series https://review.openstack.org/573604
08:51:49 giblet lyaaaaaaaarwood: which part?
08:52:00 openstackgerrit Merged openstack/nova master: libvirt: Fix the rescue race for vGPU instances https://review.openstack.org/577424
08:52:07 openstackgerrit Merged openstack/nova master: Merge server create schema for legacy BDM extension https://review.openstack.org/577152
08:52:14 openstackgerrit Merged openstack/nova stable/pike: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/563700
09:08:41 openstackgerrit Merged openstack/nova master: Use ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa in tree https://review.openstack.org/578878
09:08:55 openstackgerrit Chris Dent proposed openstack/nova master: Use nova.db.api directly https://review.openstack.org/543262
09:12:01 PapaOurs cdent: good morning
09:12:11 cdent morning
09:12:14 PapaOurs cdent: re: https://review.openstack.org/#/c/575127/2/nova/api/openstack/placement/util.py
09:12:34 PapaOurs cdent: looks like the minimum webob version we support doesn't have it
09:13:00 PapaOurs cdent: https://github.com/openstack/requirements/blob/master/lower-constraints.txt#L18
09:13:09 cdent ah, yeah, that's probably true.
09:13:31 cdent It came in with webob's big accept header refactoring
09:13:58 cdent it would be nice to raise the constraint if possible, but I could also adapt that code I suspect
09:14:51 cdent PapaOurs: what's your opinion?
09:15:55 PapaOurs cdent: I dunno
09:16:03 PapaOurs cdent: I'm not super expert on g-r
09:16:17 PapaOurs cdent: what I know is that we can at least ask for a bump on lower-reqs
09:16:42 PapaOurs but the big webob refactoring could be refraining people accepting this bump
09:17:29 PapaOurs cdent: so we could be more conservative on placement, and just try to have a backwards compatible call
09:17:32 cdent how did you discover this?
09:17:40 PapaOurs cdent: a downstream bug
09:17:56 PapaOurs cdent: since lower-reqs aren't updated, we're still on 1.7
09:18:25 cdent have you got time to make a patch?
09:18:35 cdent (if not, I can give it a go)
09:19:39 PapaOurs cdent: https://bugzilla.redhat.com/show_bug.cgi?id=1596308
09:19:39 openstack bugzilla.redhat.com bug 1596308 in openstack-nova "Placement API unexpected error: 'MIMEAccept' object has no attribute 'acceptable_offers'" [Unspecified,New] - Assigned to nova-maint
09:20:31 PapaOurs cdent: I can file an upstream bug against nova, and do the patch yeah
09:20:42 PapaOurs patch(es) even
09:20:44 cdent that would be awesome
09:20:54 PapaOurs one for lower-reqs, and one for placement
09:27:40 cdent PapaOurs: please mention the bug here once you create it
09:27:59 PapaOurs cdent: https://bugs.launchpad.net/nova/+bug/1779287
09:27:59 openstack Launchpad bug 1779287 in OpenStack Compute (nova) "placement can throw exception if webob<1.8 in use" [High,Confirmed] - Assigned to Sylvain Bauza (sylvain-bauza)
09:28:42 PapaOurs cdent: I'm firing a patch against nova
09:30:09 cdent you nick today is really throwing me off
09:32:04 PapaOurs cdent: it's my new friday nick
09:32:17 cdent what does it mean?
09:32:29 PapaOurs BearDad
09:33:25 cdent is this were slack I'd have to respond to that with some kind of emoji
09:33:39 cdent Anyway, I spent the whole first 3/4 of our conversation having no idea who you were
09:33:54 cdent thinking "oh cool, new downstream person coming upstream"
09:34:29 cdent so I am half sad and half happy
09:38:08 PapaOurs cdent: you can still whois me ;)
09:38:42 PapaOurs but sorry for the emotional lift :)
09:45:38 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
09:45:39 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
09:45:40 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
09:45:41 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
09:45:42 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
09:45:43 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
09:46:44 openstackgerrit Sylvain Bauza proposed openstack/nova master: Fix placement incompatible with webob 1.7 https://review.openstack.org/579110
09:50:22 PapaOurs cdent: finucannot: efried: https://review.openstack.org/579110
09:55:48 finucannot PapaOurs: Can we not just bump the lower-constraints on that?
09:58:43 cdent finucannot: see the backlog, there's some discussion of doing both
09:58:58 cdent s/doing/trying/ and seeing which one floats
10:02:42 kaisers Hey Novas, anyone familiar with Nova+QEMU please have a look: It seems to me the change https://review.openstack.org/#/c/553072/ introduced a dependency for QEMU v2.9+ while https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix shows QEMU v2.5 as minimum supported version. The patch only checks for a libvirt version but not for QEMU 2.9+ . Or maybe i'm overlooking something? (QEMU v2.9 adds the pass mtu to guest fe
10:03:57 kaisers I'm not absolutely sure my 2.9 ref is the required feature but it's the best lead i found so far. Our CIs are stumbling on setting the MTU as they're running QEMU 2.5 (because Xenial release)
10:11:57 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object https://review.openstack.org/573628
10:11:57 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Refactor the policies to policy https://review.openstack.org/579113
10:39:23 openstackgerrit Stephen Finucane proposed openstack/nova master: Define common variables for irrelevant-files https://review.openstack.org/578882
10:50:08 openstackgerrit Stephen Finucane proposed openstack/nova master: manage: Remove dead code https://review.openstack.org/579118
10:50:45 finucannot PapaOurs: Wanna send this through? https://review.openstack.org/#/c/530923/
10:59:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Update admin/flavors document https://review.openstack.org/573063
11:18:57 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Handle LM rollback error when detaching volumes from transient domain https://review.openstack.org/579125
11:18:59 lyaaaaaaaarwood kashyap: ^ as discussed earlier, would you mind taking a look?

Earlier   Later