| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-19 | |||
| 15:38:31 | efried | shit | |
| 15:38:54 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: tox -e fast-specs https://review.opendev.org/673356 | |
| 15:39:28 | frickler | mriedem: np, I'll keep it on my list, then. I've reduced my list of open patches to less than a screenful already with a bit of headroom even, that's enough for my summer cleanup ;) | |
| 15:53:31 | efried | mriedem: o hey mon, got a sec to help me with logstash? | |
| 15:53:53 | mriedem | yes | |
| 15:54:41 | efried | right?? | |
| 15:54:41 | efried | message:"TypeError: open: path should be string, bytes or os.PathLike, not _NormalAccessor" | |
| 15:54:41 | efried | mriedem: I was trying to logstash that kombu thing. I should be able to search for | |
| 15:55:18 | mriedem | got a link to a log with the failure? | |
| 15:55:36 | efried | https://zuul.opendev.org/t/openstack/build/369bc38f57134ece8befdf4f3d6d670e | |
| 15:57:05 | efried | mriedem: note that it's in tox-py37 (always), not sure if that's significant | |
| 15:57:34 | mriedem | since the console log is huge is it failing to publish logs to elasticsearch | |
| 15:57:35 | mriedem | ? | |
| 15:58:12 | mriedem | https://review.opendev.org/#/c/673356/1 | |
| 15:58:14 | mriedem | oops | |
| 15:58:25 | mriedem | 222.4M | |
| 15:59:16 | efried | so you mean, elasticsearch is only good if something blows up in a small way? | |
| 15:59:36 | mriedem | i wonder why job-output is not compressed | |
| 15:59:48 | mriedem | i don't see subsequent ansible task failures for compressing and publishing logs though | |
| 15:59:56 | efried | swift cutover something something | |
| 16:00:45 | mriedem | the b' is screwing with your query | |
| 16:01:23 | efried | that's going to be prevalent in py3* logs, tho | |
| 16:01:29 | mriedem | yes i know | |
| 16:01:30 | efried | And I'm not including the b'' in my search | |
| 16:01:31 | efried | should I be? | |
| 16:01:32 | mriedem | and it's always been a problem | |
| 16:01:35 | efried | o | |
| 16:01:47 | mriedem | message:"b'TypeError: open: path should be string, bytes or os.PathLike, not _NormalAccessor'" AND tags:"console" | |
| 16:01:47 | mriedem | so i'm using: | |
| 16:02:06 | mriedem | but that's not showing hits either, | |
| 16:02:12 | efried | I got hits | |
| 16:02:20 | efried | (this got resolved over the weekend, so you have to spread to last week) | |
| 16:02:30 | mriedem | i'm goign to 7 days | |
| 16:03:01 | mriedem | are you sure those are real hits? kibana has this weird thing where the first query results are always garbage | |
| 16:03:07 | efried | weird, I'm getting lots of hits with that sea... oh, 'and' is case-sensitive | |
| 16:03:07 | mriedem | and i have to hit the magnifying glass thing to re-query | |
| 16:03:25 | efried | so yeah, I'm getting nothing. | |
| 16:03:34 | mriedem | then we need to talk to clarkb | |
| 16:10:36 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Nova object changes for forbidden aggregates request filter https://review.opendev.org/671072 | |
| 16:11:21 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074 | |
| 16:12:09 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Add a new request filter to isolate aggregates https://review.opendev.org/671075 | |
| 16:13:00 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Enable request filter isolate_aggregates https://review.opendev.org/667952 | |
| 16:37:18 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 16:37:19 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 16:37:19 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 16:37:20 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: tox -e fast-specs https://review.opendev.org/673356 | |
| 16:41:20 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 16:42:04 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 16:42:05 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 16:42:05 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 16:48:54 | openstackgerrit | Adam Spiers proposed openstack/nova master: Make it easier to run a selection of tests relevant to ongoing work https://review.opendev.org/677180 | |
| 16:54:52 | openstackgerrit | sean mooney proposed openstack/nova master: Libvirt: add support for vPMU configuration. https://review.opendev.org/671338 | |
| 16:55:14 | sean-k-mooney | mriedem: ^ should address your vpmu nits. im going to grab something to eat quickly but if you see anything else leave a comment and ill fix it in an hour or so | |
| 16:55:22 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 17:01:44 | efried | Basically (IMO) we need *some* reasonable py37 coverage in there, and nova seemed as good as any. | |
| 17:01:44 | efried | mriedem: are you in favor of gating requirements on py37 using cross-nova? https://review.opendev.org/#/c/677071/5 | |
| 17:06:15 | mriedem | gmann: comments in your novaclient 2.75 change https://review.opendev.org/#/c/676275/ | |
| 17:07:43 | mriedem | is openstack-py37 voting on nova changes? | |
| 17:07:53 | mriedem | looks like yes | |
| 17:08:16 | mriedem | https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L166 | |
| 17:09:13 | mriedem | so yeah +1 | |
| 17:09:37 | efried | thanks mriedem | |
| 17:13:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: cleanup 2.75 REST API microversion history doc https://review.opendev.org/677259 | |
| 17:29:46 | openstackgerrit | Akihiro Motoki proposed openstack/nova master: PDF documentation build https://review.opendev.org/676730 | |
| 17:38:13 | openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Allow deep-linking to memory reservation section of AMD SEV spec https://review.opendev.org/677264 | |
| 18:01:12 | aspiers | err, why is there no direct link from https://docs.openstack.org/nova/latest/ to https://docs.openstack.org/nova/latest/user/ ? | |
| 18:04:03 | efried | aspiers: fix it! | |
| 18:07:39 | aspiers | efried: I was wondering if it was deliberate | |
| 18:08:14 | aspiers | "Todo | |
| 18:08:17 | aspiers | The rest of this document should probably move to the admin guide." | |
| 18:08:51 | aspiers | The top-level page directly links to some of the stuff under doc/source/user/ | |
| 18:09:21 | efried | I can't imagine why the nova doc landing page wouldn't have a link to the user landing page anyway. | |
| 18:09:45 | aspiers | It does seem odd | |
| 18:09:53 | aspiers | but I find the whole doc organisation pretty confusing | |
| 18:10:12 | efried | agree | |
| 18:10:43 | efried | At some point donnyd was on a mission to reorganize, perhaps y'all could meet minds | |
| 18:11:16 | aspiers | when I've got this damn SEV series done | |
| 18:11:33 | aspiers | I'm pretty close now, just addressing your feedback on the tip of the series and then my backlog is clear I think | |
| 18:11:47 | aspiers | that is, until the next flood of -1s come in ;-) | |
| 18:14:53 | aspiers | efried: now I'm longing for tox -e fast-docs ... | |
| 18:15:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add config_drive to 2.75 rebuild response parameters https://review.opendev.org/677267 | |
| 18:17:30 | efried | mriedem: Does this https://review.opendev.org/675908 smell like a stat mining / ATC grab exercise to you? | |
| 18:18:33 | mriedem | you mean just b/c it's nova-specs? | |
| 18:21:00 | efried | and because it's pointless, and because they put a co-author without any evidence of that person having co-authored anything, and because there's really nothing to co-author | |
| 18:21:21 | efried | basically this patch now has three contributors | |
| 18:21:50 | efried | Tempting to say, "you want a free summit ticket, go get it from somebody else's project" | |
| 18:21:51 | donnyd | efried: Still on that mission, but its hard to even get a start | |
| 18:22:09 | efried | I hear ya. | |
| 18:23:07 | donnyd | I am starting in docs with cyborg atm | |
| 18:49:51 | mriedem | efried: yeah... https://review.opendev.org/#/c/675908/3..4//COMMIT_MSG | |
| 18:50:49 | mriedem | i know "Add Python 3 Train unit tests" is a Train cycle goal - i don't really know what it is, but i know it's a goal | |
| 18:51:07 | mriedem | https://review.opendev.org/#/q/owner:%22Filippo+Inzaghi%22+status:open | |
| 18:51:27 | efried | yeah, I was okay with it at that point | |
| 18:52:03 | efried | But when Vieri, a notorious silent +1er, uploaded a new PS that added not him/her, but 98k, another notorious silent +1er, as co-author, my spidey sense went nuts. | |
| 19:00:59 | efried | dustinc: Are you busily debugging the ironic CI failure in patch #3 at this point? | |
| 19:01:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: cleanup 2.75 REST API microversion history doc https://review.opendev.org/677259 | |
| 19:01:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add config_drive to 2.75 rebuild response parameters https://review.opendev.org/677267 | |
| 19:01:55 | sean-k-mooney | efried: well give we dont run the in repo test in the nova-specs gate i dont think https://review.opendev.org/#/c/675908/4 is relly useful | |
| 19:02:39 | efried | agree, it would be just for consistency with other projects, but I really don't think it's worthwhile. | |
| 19:02:48 | mriedem | efried: welcome to my world of being annoyed with random changes like this that just look like stat padding | |