Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-06
15:28:32 openstackgerrit Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239
15:28:32 openstackgerrit Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854
15:28:33 openstackgerrit Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/510159
15:37:46 gibi cdent: I can confirm that bug based on looking at the code. It seems that the functional test did not catched it somehow
15:38:21 cdent gibi: cool, I figured you would know what was going on
15:39:24 gibi cdent: I felt save becuause of the functional coverage, but it seems we need a better test for it
15:39:32 gibi s/save/safe/
15:42:11 bauwser leakypipes: maybe tracking all the changes for https://review.openstack.org/#/c/509025/ and above would be better if we have a specless BP ?
15:42:29 bauwser leakypipes: of course, not needing a spec
15:50:30 ronlund claudiub: want to send this in? https://review.openstack.org/#/c/509766/
16:08:23 finucannot leakypipes superdan: Could you take a look at this? I don't think we need 'obj_make_compatible' functions because we're not transferring these objects over the wire, but it's gone to be sure to be sure https://review.openstack.org/#/c/508498/
16:09:44 superdan finucannot: I'm in the middle of something deep right now, but we're registering those objects which means they can go over the wire, which means they need to have the make_compat routine
16:09:54 superdan I'm sure leakypipes can speak to the over-the-wire-ness of now and future
16:10:09 openstackgerrit Merged openstack/nova master: stabilize test_resize_server_error_and_reschedule_was_failed https://review.openstack.org/509750
16:10:27 finucannot superdan: Ta. Holding for leakypipes
16:19:45 andreykurilin superdan: the first patch, where I put sort_key=["uuid"] fixed an issue. the second patch(the check for a fix at nova's side) still waits for a resources at CI
16:19:58 superdan andreykurilin: okay
16:26:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1721652 in the functional test env https://review.openstack.org/510176
16:26:38 openstack bug 1721652 in OpenStack Compute (nova) pike "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,Confirmed] https://launchpad.net/bugs/1721652
16:30:12 gibi cdent, ronlund: I started looking into the bug 1721652 but I run out of time for today. I will continue on Monday if nobody feels the urge to take it over.
16:30:14 openstack bug 1721652 in OpenStack Compute (nova) pike "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,Confirmed] https://launchpad.net/bugs/1721652
16:31:45 ronlund gibi: ok, thanks
16:36:37 melwitt what's ronlund?
16:44:50 superdan mmmmm, yeeeeeeaahhhh
16:44:50 sean-k-mooney anyone know if we were to abandon https://review.openstack.org/#/c/373293/7 would it prevent the proposal bot updating it in the future.
16:44:59 superdan that would be greeeeeeaaaat, mmmmkayyy?
16:45:50 superdan we're just going to go ahead and have to ask you to move your desk to the basement, mmmmmkayy? thaaaaaaankss...
16:46:14 openstackgerrit Merged openstack/os-vif master: Add Port Profile info to VIF objects Linux Bridge plugin https://review.openstack.org/490829
16:47:06 ronlund melwitt: ron lund is a powerful name
16:47:11 ronlund and is a name you can trust
16:47:16 ronlund for all your retirement investment needs
16:47:21 ronlund in the greater tristate area
16:47:27 melwitt lol ahh
16:47:37 sean-k-mooney finucannot: stephen is that you?
16:48:17 ronlund ron lund is a man that speaks in the 3rd person, loves turtle necks and has a great dirty blonde mustache
16:48:54 superdan oh I guess I'm wrong
16:49:12 melwitt yours is bill lumbergh
16:49:28 superdan ohhh, damn, right
16:50:00 melwitt at first I thought maybe ron lund was ron swanson from parks and rec but it wasn't
16:50:23 melwitt that's the only ron I could think of
16:53:40 ronlund i also personally know a ron bruns
16:53:43 ronlund also a powerful name
16:53:51 ronlund really anyone named "ron" shouldn't be fucked with
16:54:21 ronlund ask maya, she'll tell you
16:55:41 melwitt I dunno anyone named ron in real life
16:56:07 ronlund ron bruns is a cattleman from baltic, south dakota
16:56:17 ronlund i think he even sold insurance on the side...
16:56:27 melwitt an enterprising fellow
16:56:46 cdent sounds like ron doesn’t like taxes
16:57:37 sean-k-mooney cdent: someone likes taxes?
16:58:23 sean-k-mooney people tolerate taxes in retrun for services but i have never meet anyone who actully likes them
16:58:31 cdent well presumably anyone who cares for their fellow person in society has some small amount of like for taxes
17:00:01 cdent but ron, being a strong ron, sounds like the sort that might reject the federal gov’s right to tax
17:02:10 penick Aww whaaaat "JunoMan signed on at October 4, 2017 at 9:40:36 PM PDT and has been idle for 1 day, 12 hours, 21 minutes, "
17:02:36 openstackgerrit Elod Illes proposed openstack/nova master: WIP: Transform scheduler.select_destinations notification https://review.openstack.org/508506
17:02:51 ronlund sdague: the "should use allow passing user_data to rebuild" thread has taken a weird path,
17:03:04 ronlund sdague: effectively putting me on the fence about whether or not we should add that when removing personality from rebuild
17:03:20 superdan MrJuno: bravo
17:03:29 MrJuno Gotta own it
17:03:34 sean-k-mooney ronlund: how so? i have not been folowing it
17:03:52 ronlund sdague: it's not like it would be hard to add, and we are saying that user_data replaces personality files, and people do love their rebuild
17:04:02 ronlund sean-k-mooney: just the amount of love for rebuild
17:04:08 sdague ronlund: I'd be fine with that
17:04:31 MrJuno rybridges I think I shall decree that all members of the OpenStack team at Oath wear the scarlet letter J until we're on Ocata
17:04:56 ronlund i originally assumed that personality files were added to rebuild b/c they aren't persisted like user_data is, but looking at the change that added personality files to rebuild, there was no explanation of why in the commit message
17:05:05 ronlund it predated gerrit so i wasn't surprised by that
17:05:17 jgwentworth yeah, ppl not using floating ips like keeping their ip and volumes stay attached and all that jazz
17:08:07 cdent ronlund: so ron bruns probably feeks like he’d be even happier without that thieving corporation tax
17:08:43 ronlund i don't know his actual feelings on taxes. he's genuinely a nice guy, so i doubt it bothers him that much.
17:09:20 cdent a weak ron
17:09:43 openstackgerrit Merged openstack/nova master: Add error notification for instance.interface_attach https://review.openstack.org/506643
17:13:20 jgwentworth uh oh, I'm seeing on some gate runs of the py27 unit test job it's not running all the unit tests, only the os profiler test http://logs.openstack.org/66/509766/1/check/gate-nova-python27-ubuntu-xenial/794e1a9/testr_results.html.gz
17:13:57 jgwentworth this is bad
17:19:36 jgwentworth I think maybe it's only happening on stable
17:21:16 jgwentworth I see it on stable/pike and stable/ocata
17:22:22 jgwentworth stable/newton looks okay
17:24:25 sean-k-mooney jgwentworth: is it on master
17:24:40 jgwentworth sean-k-mooney: I'm not seeing it on master
17:25:11 sean-k-mooney jgwentworth: thats not so bad then because master should prevent anything getting backported if the unit tests fail
17:25:28 sean-k-mooney though i guess it would not catch dependecy issues
17:25:37 jgwentworth yeah, definitely not as bad as on master
17:26:18 superdan sean-k-mooney: help, but not prevent.. a backport could assert something that is true on master and not on stable and we'll think it's okay to merge
17:26:21 sean-k-mooney i guess that has something to do with the zuul v2->v3->v2 changes in the last few weeks
17:26:42 jgwentworth ronlund: I noticed on stable/pike and stable/ocata there's something wrong with our unit test jobs and they're running the os test profiler test instead of all of the unit tests ^
17:27:30 ronlund jgwentworth: sounds like an issue for mtreinish
17:28:03 ronlund did some stestr stuff get mixed up in stable?
17:28:35 sean-k-mooney jgwentworth: that should be defiend by this job spec correct https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/python-jobs.yaml#L109-L131
17:29:37 jgwentworth I dunno, I'm not familiar with how this works but that looks like probably
17:30:18 ronlund the zuulv3 jobs were defined elsewhere
17:30:21 ronlund in openstack-zuul-jobs
17:30:29 ronlund but that's for zuulv3, the zuulv2 stuff should be as before
17:30:30 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate allowed_direct_url_schemes and nova.image.download.modules https://review.openstack.org/510195
17:30:32 ronlund but i'm no expert
17:30:52 ronlund i would be suspect of something with stestr but we shouldn't be using that in stable
17:31:49 ronlund cdent: btw i got a recreate of the scheduling 409 failure on https://review.openstack.org/#/c/507918/
17:32:01 ronlund http://logs.openstack.org/18/507918/6/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/d4f175d/logs/screen-n-sch.txt.gz#_Oct_04_18_32_08_753794
17:32:15 ronlund it doesn't have a logging patch in that run which logs which instance failed and prompted retries
17:32:24 ronlund but maybe not necessary to debug?
17:32:32 jgwentworth newton: https://review.openstack.org/#/c/509441/ ocata: https://review.openstack.org/#/c/509440/ pike: https://review.openstack.org/#/c/509439/
17:32:53 sean-k-mooney looking at https://review.openstack.org/#/c/509766/ its reporting as jenkins not zuul so i guess the python job is running as zuul v2.5 not v3

Earlier   Later