| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-06-14 | |||
| 14:21:36 | cdent | they suck | |
| 14:21:40 | dansmith | bummer :( | |
| 14:30:59 | artom | sahid, the claim would have to done on the destination once we're sure the migration is really happening (ie, after all the checks), right? | |
| 14:32:08 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add status and links fields to version document at / https://review.openstack.org/575117 | |
| 14:32:45 | sahid | artom: hum... which checks are you talking? the claim should be done just after that the scheduling find a destination | |
| 14:33:06 | sahid | scheduler | |
| 14:33:24 | artom | sahid, dammit, I need to find the actual functions how | |
| 14:33:25 | artom | ;) | |
| 14:33:36 | artom | The _check_can_live_migrate stuff | |
| 14:34:24 | artom | I'm asking because in the resource tracker, we do _claim_existing_migration if a Migration already exists, and set the dest node to "ourselves" | |
| 14:34:38 | artom | So, presumably, we have to be sure we're actually doing to receive the instance | |
| 14:35:02 | mtreinish | stephenfin, efried: is stestr run --failing what you're looking for? https://stestr.readthedocs.io/en/latest/MANUAL.html#running-previously-failed-tests | |
| 14:35:06 | artom | And that we're not going to fail a check right after claiming the existing migration | |
| 14:35:38 | openstackgerrit | Dan Smith proposed openstack/nova stable/pike: Change consecutive build failure limit to a weigher https://review.openstack.org/573248 | |
| 14:35:46 | stephenfin | mtreinish: It would appear so. Excellent | |
| 14:35:54 | stephenfin | mtreinish: Thanks :) | |
| 14:36:53 | mtreinish | sure, np | |
| 14:38:49 | mriedem | well look who decides to show up | |
| 14:38:58 | mriedem | when testr talk is happening but not when the gate is burning down | |
| 14:39:40 | mtreinish | heh, well complain to my mgt I had to sit in a booth all day yesteday to talk about IBM cloud | |
| 14:40:18 | mtreinish | although, the funny thing is I only got questions about AIX and Z. no one cared about the cloud | |
| 14:40:26 | mriedem | AIX? | |
| 14:40:31 | mriedem | i could maybe understand Z | |
| 14:40:57 | mtreinish | apparently people are still using aix, I was very confused too | |
| 14:41:09 | mriedem | and as400, die hards | |
| 14:41:11 | efried | Because it is still awesome. | |
| 14:41:12 | dansmith | mtreinish: all those people that bought 25 year licenses | |
| 14:41:24 | cdent | second worst unix imp | |
| 14:41:33 | dansmith | cdent: behind Xenix? | |
| 14:41:36 | efried | or prefer price/performance to simply price | |
| 14:41:40 | sahid | artom: hum not really sure to follow sorry, claim is doing some checks and then update the resources so we should never be in a sitatuon where a check fails after we claimed the resources | |
| 14:41:41 | cdent | well maybe third then | |
| 14:41:43 | dansmith | hah | |
| 14:41:50 | cdent | fourth | |
| 14:42:00 | cdent | shit, there are a lot of horrible unix implementations | |
| 14:42:06 | dansmith | including windows | |
| 14:43:34 | sahid | artom: look at conductor/tasks/live_migrate.py at this point the scheduler (or user) selected a destination | |
| 14:44:55 | mriedem | cdent: i always thought that song was about weed, but i'm not so sure | |
| 14:44:56 | sahid | there is a rpcapi check_can_live_migrate_destination. so it's here we you we have to claim the resources | |
| 14:45:17 | cdent | it's a metaphor of at least two things, including weed | |
| 14:45:21 | jangutter | Re: AIX, 15 years ago at Uni, I did a bit of tech support on one of our top researcher's AIX workstation. He did everything in TeX. He also had no subdirectories. He knew every filename by heart. | |
| 14:45:29 | jangutter | that's my memory of AIX. | |
| 14:45:59 | cdent | jangutter: that sounds like every memory of AIX I've ever heard | |
| 14:46:13 | mriedem | i think my dad knew every filename on his desktop by heart because that's where every file went | |
| 14:46:18 | mriedem | miles and miles of desktop | |
| 14:46:35 | dansmith | my dad makes all the important files named in ALL CAPS | |
| 14:46:43 | mriedem | at least he has a system | |
| 14:46:46 | mriedem | *SYSTEM | |
| 14:46:52 | dansmith | which means he has a directory full of files with ALL CAPS names | |
| 14:46:59 | dansmith | because they're all important when you create them | |
| 14:46:59 | mriedem | i had a girlfriend in college that tapped in all caps, | |
| 14:47:05 | mriedem | sounded like she was yelling at me constantly | |
| 14:47:05 | dansmith | um | |
| 14:47:12 | mriedem | *typed | |
| 14:47:14 | dansmith | oh | |
| 14:47:34 | mriedem | it was a short and painful relationship | |
| 14:47:42 | dansmith | you mean PAINFUL | |
| 14:47:49 | cdent | mriedem: the song is also about being on the dole | |
| 14:47:59 | dansmith | mriedem: I slapped that ironic host manager fix into the pike backport | |
| 14:48:21 | dansmith | will experimental run an ironic job on that? | |
| 14:48:28 | mriedem | we have a check queue ironic job | |
| 14:48:30 | mriedem | non-voting | |
| 14:48:39 | dansmith | I didn't see it fail | |
| 14:49:03 | mriedem | yeah... | |
| 14:49:07 | mriedem | http://logs.openstack.org/48/573248/4/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/c7bd653/ | |
| 14:49:10 | mriedem | what gives | |
| 14:49:27 | dansmith | was it on resource classes already? | |
| 14:49:54 | mriedem | we're not using the ironic host manager apparently in that job http://logs.openstack.org/48/573248/4/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/c7bd653/logs/etc/nova/nova.conf.txt.gz | |
| 14:50:00 | mriedem | but i bet tripleo or someone else | |
| 14:50:00 | dansmith | yeah | |
| 14:50:01 | mriedem | *is | |
| 14:50:12 | mriedem | which is how tonyb was beaten over the head | |
| 14:50:18 | dansmith | experimental runs tripleo right? | |
| 14:50:46 | mriedem | hold dear caller | |
| 14:51:31 | mriedem | don't think so, unless that's this http://logs.openstack.org/48/573248/4/check/ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa/c7bd653/logs/etc/nova/nova.conf.txt.gz | |
| 14:51:34 | mriedem | but it doesn't say tripleo | |
| 14:51:39 | edleafe | cdent: uh, no, it isn't. http://www.songfacts.com/detail.php?id=397 | |
| 14:52:08 | mriedem | gdi wrong link | |
| 14:52:20 | mriedem | dansmith: answer is i don't know | |
| 14:52:23 | dansmith | mriedem: hmm, yeah, not running a tripleo job | |
| 14:52:27 | dansmith | I just started experimental | |
| 14:53:45 | cdent | edleafe: on the weed or the dole? because here in the uk "everyone" acknowledges it is both, because of the kutchie to dutchie doubling | |
| 14:56:49 | sahid | mriedem, mdbooth_: if you can have a look at this when you have a moment https://review.openstack.org/#/c/570528/ | |
| 14:56:59 | edleafe | cdent: How does a cooking pot relate to being on the dole? | |
| 14:57:01 | mriedem | jmlowe: so what was the https issue with nova client? | |
| 14:57:09 | cdent | edleafe: sharing | |
| 14:57:16 | sahid | this is half fixing an issue, the real fix that would handle all the case is a bit more complicated | |
| 14:57:30 | sahid | wanted to get you idea | |
| 14:57:38 | jmlowe | It wouldn't parse the endpoint correctly to pick up on https unless there was a trailing slash | |
| 14:58:22 | jmlowe | /v2.1 doesn't work /v2.1/ does | |
| 14:59:09 | cdent | edleafe: important to remember that any pop song up until the maybe late 80s that can't otherwise be explained is probably about the dole: https://www.youtube.com/watch?v=RZ2oXzrnti4 (of course that one is pretty easy to explain) | |
| 14:59:34 | cdent | s/pop song/pop song in the uk/ | |
| 15:02:22 | mriedem | sahid: i don't like that patch ,see comments inline | |
| 15:03:02 | jmlowe | mriedem: oddly osc does parse correctly nova cli doesn't | |
| 15:06:34 | mriedem | jmlowe: ok i thought that was resolved in the patch i linked from andreykurilin yesterday, but apparently not | |
| 15:06:38 | mriedem | b/c i'm pretty sure you had that fix | |
| 15:06:40 | kashyap | mriedem: Yeah, I agree with your comment here: https://review.openstack.org/#/c/570528/5//COMMIT_MSG@17 | |
| 15:07:38 | jmlowe | mriedem: I did, however I'll probably need the workaround in place forever, once a broken client gets out it's nearly impossible to fix | |
| 15:07:41 | mriedem | kashyap: the only problem is, in check_can_live_migrate_destination, i don't think we actually *know* the source instance image backend | |
| 15:08:22 | kashyap | mriedem: Nod; yeah, I was going to look into that code | |