| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-26 | |||
| 15:09:14 | mriedem | oh actually it's just the join() i htink | |
| 15:09:22 | mriedem | .options(joinedload('cell_mapping')) will load up the null attribute | |
| 15:09:55 | dansmith | I meant "wouldn't" | |
| 15:10:09 | dansmith | I would think you could still query by that column directly, not sure why you would need to join the other | |
| 15:10:36 | dansmith | but no, nothing really stands out to me | |
| 15:11:01 | aspiers | hrm, the first of my SEV patches has bizarrely just started failing openstack-tox-lower-constraints due to having an old os-traits which misses the SEV trait | |
| 15:11:06 | mriedem | i run it through _from_db_object which loads the attribute on the ovo | |
| 15:11:13 | mriedem | get a lazy-load session error otherwise | |
| 15:11:18 | mriedem | but yeah it was the join() that was filtering it out, fixed | |
| 15:11:19 | mriedem | thanks | |
| 15:11:31 | aspiers | can I just up the version in lower-constraints? has something recently changed with the way lower-constraints is tested? | |
| 15:12:00 | mriedem | aspiers: if you require a new min of os-traits then yes you need to bump the version in lower-constraints.txt | |
| 15:12:10 | aspiers | mriedem: in the same commit? | |
| 15:12:16 | kashyap | mriedem: If you get a minute, when backporting this fix of yours to Rocky (admittedly, downstream), any other patch is required? -- https://review.opendev.org/#/c/591607/ ("Force refresh instance info_cache during heal") | |
| 15:12:44 | mriedem | jesus | |
| 15:12:57 | mriedem | i got this same questoin from sean-k-mooney earlier about some other patch going to newton | |
| 15:13:12 | openstackgerrit | Merged openstack/nova master: Create request spec, build request and mappings in one transaction https://review.opendev.org/586742 | |
| 15:13:33 | mriedem | kashyap: yes, there is an online data migration below that in the series, | |
| 15:13:42 | openstackgerrit | Eric Fried proposed openstack/nova master: Hacking N363: Don't use spec[_set]='string' https://review.opendev.org/650370 | |
| 15:13:42 | aspiers | kashyap: not sure if it helps here, but in general https://github.com/aspiers/git-deps can help answer that question | |
| 15:13:46 | mriedem | and that online data migration has resulted in a couple of regressions in stein still being worked | |
| 15:14:03 | aspiers | kashyap: https://github.com/aspiers/git-deps/blob/master/USE-CASES.md#use-case-1-porting-between-branches | |
| 15:14:47 | kashyap | aspiers: Ah, thanks. I was doing a lame `git blame -C -M -L [...]`, and nothing turned up besides Matt's commit | |
| 15:14:55 | mriedem | kashyap: http://lists.openstack.org/pipermail/openstack-discuss/2019-April/005242.html | |
| 15:15:04 | kashyap | mriedem: /me clicks; thank you, sir | |
| 15:15:23 | mriedem | aspiers: yes same commit | |
| 15:15:25 | kashyap | And actually, thank _you_ for the work. Someone downstream is bugging me to backport it | |
| 15:15:31 | aspiers | mriedem: cool thanks | |
| 15:16:07 | kashyap | mriedem: Whatever beverage (you didn't take up my Berlin "root beer" offer) you like in Denver. (Maybe they have a "user beer" :D) | |
| 15:16:09 | mriedem | kashyap: you can thank Maciej Jozefczyk since he did most of the work and the online data migration | |
| 15:16:21 | kashyap | Ah, thanks to him, too, from OVH | |
| 15:16:23 | mriedem | what is that now, like a 6 pack you owe me? | |
| 15:16:37 | kashyap | Guitly as charged. I'll shell it out in Denver | |
| 15:16:39 | mriedem | dtroyer has also said he owes me a few, i'll have rot gut | |
| 15:17:00 | kashyap | mriedem: You got an Amazon Wishlist (that doesn't contain things like Hubble Telescope or something like that) :D | |
| 15:17:54 | mriedem | heh, yeah but don't really want to get into that game | |
| 15:18:04 | mriedem | nor share my dirty wishlist secrets | |
| 15:18:09 | kashyap | Hehe | |
| 15:18:24 | kashyap | But I hate how much I depend on Amazon. But no one can beat their prices. Damn them | |
| 15:18:46 | kashyap | (I still go and support the local stores as much as I can.) | |
| 15:18:46 | aspiers | +1 on that | |
| 15:19:24 | kashyap | Although I live in Gent, I sometimes order from the Netherlands-based "bol.com" (a mini Amazon, as it were), but at least 5-6 euros more expensive | |
| 15:20:31 | mordred | kashyap: I have very conflicting feelings about my retail reliance on amazon ... because on the one side, amazon. but on the other side, I'm not sure that their competitors in the space are *better* | |
| 15:20:35 | kashyap | aspiers: Thanks for the interesting 'git-deps' URL. (Bookmarked, I'll check it out "soon".) | |
| 15:20:52 | aspiers | kashyap: the video is only a few mins :) | |
| 15:20:58 | kashyap | mordred: Yeah. But "the competitor is not better" is not a very satisfying argument | |
| 15:21:07 | mordred | no, it's not | |
| 15:21:45 | kashyap | It's just obscene the way they under-cut & squish everything else. Like a 9000-pound blue whale | |
| 15:21:50 | aspiers | kashyap: I guess you missed the L&L I gave on it in Denver last year https://www.youtube.com/watch?v=f6anrSKCIgI | |
| 15:21:51 | mordred | yup | |
| 15:21:55 | kashyap | (Okay, okay, I'll get off my high horse) | |
| 15:22:10 | mordred | kashyap: to be fair - a 9000-pound blue whale would actually be pretty awesome | |
| 15:22:35 | kashyap | aspiers: I didn't go to any other Denver event. This is the first time for me. (I make it to at most one OpenSt^H^H^OpenInfra event a year) | |
| 15:22:49 | aspiers | kashyap: ah ok | |
| 15:23:25 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add new "supports_amd_sev" capability to libvirt driver https://review.opendev.org/638680 | |
| 15:25:28 | kashyap | mdbooth: Heh. (I'm actually liking my "pivot" to "buy as much locally as you can, even if it's a tad more expensive". Although, I'm in a very convenient "city town" of 250K people, where I can get most things done via biking. I know that my fellow folks in the US (given its huge surface area) don't have the same convenience | |
| 15:26:01 | efried | stephenfin: What's your schedule for the PTG? Are you around all three days? Which bits are consumed with non-nova? | |
| 15:28:51 | kashyap | efried: For mine, you asked the other day -- if it's still possible, still got a slot on Saturday morning to move the Secure Boot? Sorry for the re-shuffle | |
| 15:29:10 | kashyap | (I won't cry into my pillow if you can't move it.) | |
| 15:29:10 | efried | looking | |
| 15:29:38 | efried | kashyap: 1045 Saturday morning? | |
| 15:29:45 | kashyap | Sure | |
| 15:29:48 | kashyap | Thank you! | |
| 15:30:06 | efried | kashyap: 15 minutes? | |
| 15:31:42 | gibi_off | mriedem: problem reproduced with the lastest patch set but the stack trace is still too long http://logs.openstack.org/68/654468/7/check/openstack-tox-py36/1c4087e/job-output.txt.gz#_2019-04-26_15_22_42_143486 | |
| 15:32:41 | kashyap | efried: Yeah, should suffice. | |
| 15:32:46 | efried | k | |
| 15:32:49 | kashyap | I normally don't digress :D | |
| 15:32:52 | mdbooth | efried: I believe stephenfin is travelling today. | |
| 15:32:56 | kashyap | (If I may say so myself.) | |
| 15:32:57 | efried | kashyap: It's hardly you I'm worried about | |
| 15:33:10 | efried | mdbooth: ack, thanks. | |
| 15:35:52 | kashyap | efried: Guess you're sharpening your "no-ratholing" knife :D | |
| 15:36:09 | efried | kashyap: I'm bringing special choke holds | |
| 15:36:16 | kashyap | Haha | |
| 15:36:22 | kashyap | Sounds ominous. | |
| 15:37:00 | kashyap | "Shime-waza", I see. | |
| 15:38:19 | mriedem | gibi_off: hmm, wth | |
| 15:38:53 | efried | kashyap: ah, judo. The fact that they can enumerate their chokes... | |
| 15:38:54 | mriedem | i'm unable to set the recursion depth limit below 55, so bumped it to 100 | |
| 15:39:25 | mriedem | but i wonder why the _get_counter in oslo.config isn't raising an error, unless the cache is getting reset along with our counter and that's why we continue to recurse | |
| 15:40:45 | gibi_off | mriedem: based on the stack trace it is now recursing on the code we added for trobleshooting https://review.opendev.org/#/c/654466/6/oslo_config/cfg.py@2627 | |
| 15:40:46 | mriedem | methinks the cache is always getting reset | |
| 15:41:31 | mriedem | ha | |
| 15:41:32 | mriedem | sob | |
| 15:42:58 | gibi_off | accessing self._get_counter calls __getattr__ that calls back to _get | |
| 15:43:10 | mriedem | yup | |
| 15:43:16 | gibi_off | sh*t | |
| 15:43:57 | mriedem | so __getattr__ just needs to check for '_get_counter' and return it directly | |
| 15:43:59 | mriedem | rather than recurse | |
| 15:44:00 | mriedem | yeah? | |
| 15:44:05 | gibi_off | mriedem: yeah | |
| 15:44:17 | mriedem | updating | |
| 15:44:28 | gibi_off | mriedem: but then I don't get why the self.__cache cause the same thing | |
| 15:44:53 | mriedem | doesn't cause the same thing? | |
| 15:44:59 | mriedem | maybe it is... | |
| 15:45:04 | mriedem | and maybe that's the bug | |
| 15:45:09 | gibi_off | mriedem: doesn't call __getattr__ | |
| 15:45:12 | gibi_off | ahh | |
| 15:45:30 | gibi_off | maybe that is the bgu | |