Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-13
19:17:20 artom Yeah
19:17:26 artom Or even with side_effect
19:17:56 artom Something like side_effect=[original_fill] + itertools.repeat(raise)
19:17:59 sean-k-mooney it is a side effect
19:18:07 sean-k-mooney that is how the fuction is used
19:18:20 sean-k-mooney *stub_fuction
19:18:36 artom Right, I meant... we don't need the weirdly scoped function introspecting itself thing
19:18:38 sean-k-mooney with all that said what mriedem worte appeares to work
19:18:46 artom Not that it's not clever
19:18:59 artom It'll just cause headaches to whoever has to work with it next
19:26:16 artom o/ see ya Monday
19:27:44 sean-k-mooney oh i see what its doing its filling the provider mapping if tis not filled
19:28:25 sean-k-mooney and raise an exception if called again
20:51:31 mriedem sean-k-mooney: heh, of course now the libvirt logging patch failed in the gate b/c of the functional notifications patch
20:54:24 sean-k-mooney nothing else would do sure :) i has to keep up the tradtion of the rest
20:56:58 mriedem corvus is going to promote both
20:59:03 sean-k-mooney if they fail again we might need to squash them. the failures are not determisitic so we should be able to merge them on there own but we might not want to retry them that much
21:09:43 efried o/
21:09:45 efried checking in
21:09:57 efried I assume we're holding rechecks until we can merge some of those gate fixes mriedem?
21:10:09 efried I've got eight or so lined up.
21:10:15 efried rechecks, not fixes
21:11:26 mriedem efried: yeah
21:11:31 mriedem corvus just promoted to top of gate
21:12:06 efried nice of him
21:12:37 mriedem i think i just bring out the best in people
21:12:39 mriedem you know?
21:12:50 efried threats of violence will do that
21:12:53 mriedem ha
21:13:34 efried do you want me to merge https://review.opendev.org/#/c/682140/ ?
21:13:43 efried despite artom hatin on it
21:14:04 efried even though you spelled stacktrace wrong
21:17:26 sean-k-mooney he did. unsurpisingly i did not notice
21:18:28 mriedem i got it correct in one spot
21:18:30 mriedem i was rushing
21:18:54 mriedem it's up to me if you want to approve it or let it sit, it's not the big blow up like the libvirt host capabilities one
21:20:02 mriedem i think both your alternative and artom's are equally gross
21:20:04 sean-k-mooney ya that one does not actully casuse the test to fail and the log mesage is like 20 lines vs 100s
21:20:18 efried how many times do we actually need to raise?
21:21:09 sean-k-mooney well once really. the stub raises if its called more then once
21:21:20 sean-k-mooney i doubt we will call it again if we raise
21:22:25 sean-k-mooney i +1'd it because a.) it works and b.) while it was surpising at first glance i understand how it works after looking at it for 30 seconds
21:22:54 mriedem you can't use wraps because we need to side effect the error in the 2nd call,
21:23:08 sean-k-mooney ya you cant
21:23:10 sean-k-mooney i tried
21:23:12 mriedem and we can't just mock it because the test fails if you don't actually call the original method on the first pass
21:23:27 sean-k-mooney yep
21:23:40 mriedem and that was the bug i uncovered
21:24:35 sean-k-mooney out of interest why do you dislike moveing the stub function inside the mock context manager
21:25:14 sean-k-mooney functionally its identiacla to what you wrote but im just wondering out of interest
21:28:50 efried I approved it.
21:29:40 mriedem sean-k-mooney: but it's late on a friday and i don't feel like updating the patch :)
21:29:46 mriedem and i'm trying to do some education stuff
21:29:51 mriedem *b/c
21:30:07 sean-k-mooney that totally fair
21:30:28 sean-k-mooney i was just wondering if you had a technical reason i missed
21:31:10 sean-k-mooney as i said i +1 because i was ok with what you wrote and didnt think it was really worth a respin
21:32:36 efried okay, unless there's something else I can do here, I'm going to bugger back off and just monitor the gate
21:32:50 efried looks like we're back up to a 12h delay \o/
21:48:07 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Fix a misuse of assertGreaterEqual https://review.opendev.org/682168
22:24:33 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: neutron: refactor nw info cache refresh out of associate_floating_ip https://review.opendev.org/682181
22:24:34 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Find instance in another cell during floating IP re-association https://review.opendev.org/682183
22:24:34 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Trap and log errors from _update_inst_info_cache_for_disassociated_fip https://review.opendev.org/682182
22:28:18 mriedem gibi: just so you're aware we're trying to get https://review.opendev.org/#/c/682133/ and https://review.opendev.org/#/c/682025/ through the gate since things keep failing on those bugs
#openstack-nova - 2019-09-14
00:09:40 openstackgerrit Merged openstack/nova master: libvirt: stub logging of host capabilities https://review.opendev.org/682133
00:09:49 openstackgerrit Merged openstack/nova master: Fix a misuse of assertGreaterEqual https://review.opendev.org/682025
00:43:54 mriedem_afk let the rechecks begin
00:44:09 sean-k-mooney i was just about to saw the same
00:44:21 sean-k-mooney *say
00:45:24 sean-k-mooney if it was any it would be somewhere in the mid atlantic i think
00:45:31 tbachman lol
00:45:51 tbachman No matter which TZ I’m in, I see you online
00:45:58 tbachman #botcheck sean-k-mooney
00:46:19 sean-k-mooney also im currently watching youtube and playing eve online. i just hapen to have irc open on my laptop
00:46:24 tbachman heh
01:15:25 sean-k-mooney efried_afk: gibi by the way mriedem has rechecked the numa series. we need to recheck the vpmem series and cpu serise but we proably should wait a while to see if the fixes above help things merge in the gate
01:15:59 sean-k-mooney i have not checked gibi series but i assume there are patces pending for bandwith migration also
01:17:10 sean-k-mooney actully looks like gibi's bandwith serise is currenly in gate :)
05:09:19 openstackgerrit Merged openstack/nova master: Make SRIOV computes non symmetric in func test https://review.opendev.org/681667
05:09:57 openstackgerrit Merged openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
05:10:06 openstackgerrit Merged openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
05:10:16 openstackgerrit Merged openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019
05:10:25 openstackgerrit Merged openstack/nova master: Extract pf$N literals as constants from func test https://review.opendev.org/680991
05:10:35 openstackgerrit Merged openstack/nova master: Improve dest service level func tests https://review.opendev.org/680998
05:27:15 openstackgerrit Merged openstack/nova master: Follow up for Ib50b6b02208f5bd2972de8a6f8f685c19745514c https://review.opendev.org/681490
05:27:23 openstackgerrit Merged openstack/nova master: Skip querying resource request in revert_resize if no qos port https://review.opendev.org/681513
05:27:37 openstackgerrit Merged openstack/nova master: Follow up for I220fa02ee916728e241503084b14984bab4b0c3b https://review.opendev.org/680810
07:05:21 openstackgerrit Brin Zhang proposed openstack/nova master: Fix legacy issues in Filter migrations by user_id/project_id https://review.opendev.org/682198
07:25:17 brinzhang_ The failure rate of Zuul in these two days is low
07:25:30 brinzhang_ so many ptach were failed
13:40:44 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: libvirt: stub logging of host capabilities https://review.opendev.org/682210
15:37:42 donnyd is there a method in nova to tell it to boot an instance when the hypervisor comes up? So for instance if I needed to do maintenance, but wanted specific instances to come up automatically
15:37:50 donnyd Is there a way to do that in Nova
16:37:58 openstackgerrit Merged openstack/nova master: PDF documentation build https://review.opendev.org/676730
17:29:28 openstackgerrit Merged openstack/nova master: Use SpawnIsSynchronousFixture in reschedule functional tests https://review.opendev.org/682140
22:43:57 sean-k-mooney donnyd: i dont know if you found it our not but you can use https://docs.openstack.org/nova/rocky/configuration/config.html#DEFAULT.resume_guests_state_on_host_boot
#openstack-nova - 2019-09-15
01:34:30 openstackgerrit Merged openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074
02:57:30 openstackgerrit Merged openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827
02:57:48 openstackgerrit Merged openstack/nova master: LM: Use Claims to update numa-related XML on the source https://review.opendev.org/635229
05:34:15 openstackgerrit Merged openstack/nova master: Note about Destination.forbidden_aggregates https://review.opendev.org/680945

Earlier   Later