| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-06 | |||
| 14:08:13 | andreykurilin | superdan: superdan: made a patch to check sort_keys, but based on the queue of the zuul the results will be in several hours | |
| 14:11:01 | andreykurilin | superdan: should I create a bug report? | |
| 14:11:14 | superdan | andreykurilin: yeah please create a bug and I'll some debugging | |
| 14:11:19 | superdan | sorry, I'm stuck on a call atm | |
| 14:11:24 | andreykurilin | thanks | |
| 14:21:57 | andreykurilin | superdan: https://bugs.launchpad.net/nova/+bug/1721791 | |
| 14:21:58 | openstack | Launchpad bug 1721791 in OpenStack Compute (nova) "Pagination of instances works incorrect" [Undecided,New] | |
| 14:22:34 | superdan | andreykurilin: thanks, I'm not sure how this is happening, but I'm really distracted on this call | |
| 14:22:39 | superdan | andreykurilin: are you going to be around for a while? | |
| 14:23:03 | andreykurilin | np, I'll planning to be there :) | |
| 14:30:20 | openstackgerrit | Merged openstack/nova master: Remove dest node allocations during live migration rollback https://review.openstack.org/507687 | |
| 14:33:38 | superdan | oooh, I might have a recreate | |
| 14:34:09 | cdent | must be because you’re super | |
| 14:35:15 | superdan | andreykurilin: do you do a regular unpaged list after the fail at all? | |
| 14:35:30 | superdan | andreykurilin: I kinda feel like one of the instances has to be in ERROR state, in cell0 to make this happen | |
| 14:38:57 | andreykurilin | superdan: while doing unpaged list, both instances are returned. After making a boot request, we are fetching the status of VM and do not continue until it become ACTIVE. Both VMs returned ACTIVE status | |
| 14:39:18 | andreykurilin | so I'm pretty sure that they are not in ERROR while listing | |
| 14:40:08 | superdan | andreykurilin: okay they should be sorted by created_at,id which is stable if you only have one database. Unless you have multiple cells here, or instances in cell0, I'm not sure how you could end up with unstable sort | |
| 14:41:59 | andreykurilin | superdan: it is dsvm job with a single node. it doesn't have any special configs | |
| 14:42:12 | superdan | yeah | |
| 14:42:26 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP Always put 'uuid' into sort_keys for stabile instance lists https://review.openstack.org/510140 | |
| 14:42:35 | superdan | andreykurilin: can you try with this in place ^ ? | |
| 14:43:02 | superdan | if you revert the functional part of that change, the test added fails in the same way | |
| 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: 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 | |