| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-06 | |||
| 14:43:53 | andreykurilin | ok, will make a depends on patch | |
| 14:51:34 | andreykurilin | superdan: btw, performance of list action is quite good. before I added a limit to the loop, debug messages flooded the log file by 10gb of text (until jenkins kicked the job by temout) :D | |
| 14:51:53 | superdan | andreykurilin: hah, cool | |
| 15:04:27 | superdan | andreykurilin: hmm, actually, that test isn't fully stable, so I need to keep working on it | |
| 15:05:08 | mriedem | fried_rice: merry friday https://review.openstack.org/#/c/488137/22 | |
| 15:05:15 | mriedem | i didn't -1, but i'm sort of inlined to | |
| 15:05:24 | fried_rice | mriedem ack, looking. | |
| 15:13:51 | cdent | ronlund: nobody wants to love on your doc fix? https://review.openstack.org/#/c/502168/ | |
| 15:17:28 | cdent | fried_rice: you still have https://review.openstack.org/#/c/499826/ in your mind? what do we need to do to resolve that? | |
| 15:17:43 | fried_rice | ... | |
| 15:18:11 | cdent | i hear that | |
| 15:18:21 | fried_rice | cdent I actually keep forgetting to put it on the "stuck reviews" list for the nova meetings. | |
| 15:18:45 | fried_rice | It really seems like overkill to put up a whole microversion for that change. | |
| 15:19:00 | cdent | yah | |
| 15:19:02 | fried_rice | But the process nazis would freak out if we slid it into some unrelated change that's doing a legit microversion bump. | |
| 15:19:47 | fried_rice | And I *totally* have no problem doing it without a microversion bump. If the rules forbid such a change, the rules are silly. | |
| 15:19:51 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | |
| 15:20:00 | fried_rice | But I clearly don't get to make that call. | |
| 15:20:49 | fried_rice | cdent Makes it harder that we cut a release since the original splitup was done. | |
| 15:21:16 | fried_rice | you know, the one that didn't cut a new microversion when it changed the API in a similar (but more extensive) way than this. | |
| 15:22:04 | fried_rice | cdent Guess I'll add it to "stuck reviews" now while I'm thinking about it, and we can discuss it next Thursday. | |
| 15:22:18 | cdent | an astute plan | |
| 15:22:33 | fried_rice | cdent Thanks for the reminder | |
| 15:22:47 | cdent | was doing my weekly cruise of placement tagged bugs | |
| 15:25:52 | cdent | gibi: you seen https://bugs.launchpad.net/nova/+bug/1721652 ? references a change you made as the potential cause | |
| 15:25:53 | openstack | Launchpad bug 1721652 in OpenStack Compute (nova) "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [Undecided,New] | |
| 15:27:33 | gibi | cdent: looking... | |
| 15:28:31 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add alternate hosts https://review.openstack.org/486215 | |
| 15:28:32 | openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | |
| 15:28:32 | openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | |
| 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 | 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:50 | superdan | mmmmm, yeeeeeeaahhhh | |
| 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 | |