| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-19 | |||
| 14:35:58 | sean-k-mooney | aspiers: no you just add words that relate to the thing your doing when you type it | |
| 14:36:00 | aspiers | right, except that will take like 20-30s longer because a) it goes through tox and b) it does test discovery | |
| 14:36:24 | sean-k-mooney | if i change ovo i add objecst if i modify the libvirt dirver i add libvirt | |
| 14:36:24 | aspiers | sean-k-mooney: sure but that's a different use case | |
| 14:36:48 | aspiers | I'm not proposing to replace any existing workflows, just add the option of a quick and easy extra one | |
| 14:36:55 | sean-k-mooney | right but if i want to run just one test it add teh test name or use an ide. | |
| 14:37:05 | aspiers | it's already saved me a bunch of typing this morning since I wrote it | |
| 14:37:19 | sean-k-mooney | anyway it nice to have a script for the new stuff | |
| 14:37:33 | sean-k-mooney | just i dont want it to imply that it will run all the relevent tests | |
| 14:39:46 | sean-k-mooney | aspiers: its good for saving time locally as long as it does not encurage people to waste gate time but not running the test they should have locally | |
| 14:40:00 | openstackgerrit | Adam Spiers proposed openstack/nova master: Switch some GitHub URLs to point to opendev.org https://review.opendev.org/677185 | |
| 14:40:37 | aspiers | sean-k-mooney: I'm not sure how providing another way to run tests locally would discourage people from running tests locally ;-) | |
| 14:41:02 | sean-k-mooney | aspiers: if that is all they run they will miss test they should have run | |
| 14:41:17 | aspiers | tools are never gonna be able to stop developers from being stupid | |
| 14:41:41 | sean-k-mooney | yes but it can make it easier for them to be :) | |
| 14:41:48 | aspiers | sure but at some point you just have to trust people to use their brain | |
| 14:41:54 | aspiers | and this is not rocket science | |
| 14:42:10 | aspiers | noone is saying "OK now we have this new tool and that is the ONLY tool you should eer use" | |
| 14:42:14 | aspiers | ever | |
| 14:42:57 | sean-k-mooney | i know but even with the regex approch its rare but sometime if i dont do a full tox run i will miss a test that the gate finds | |
| 14:43:06 | aspiers | I'd also argue that developer time is more expensive than gate time, but that's a whole other debate | |
| 14:43:15 | openstackgerrit | Eric Fried proposed openstack/nova master: Tests: autospecs all the mock.patch usages https://review.opendev.org/470775 | |
| 14:43:21 | aspiers | yeah, so use your regex approach | |
| 14:43:43 | aspiers | or, use my new approach while you are in the middle of coding because it's quicker, then use yours when you think you are done and ready to upload | |
| 14:43:56 | aspiers | doing both gets the best of both worlds | |
| 14:44:17 | aspiers | this is just basic 80/20 rule application | |
| 14:44:36 | efried | aspiers: this bud's for you https://review.opendev.org/#/c/470775/42/nova/tests/unit/test_json_ref.py@58 | |
| 14:45:57 | sean-k-mooney | ya i get that. i like fast8 for example but that is always correct. | |
| 14:46:02 | aspiers | efried: you mean you want me to submit a separate fixup later? | |
| 14:46:20 | aspiers | sean-k-mooney: running a single test is always correct, it's not just complete :) | |
| 14:46:24 | aspiers | this is no different to fast8 | |
| 14:46:30 | aspiers | fast8 is not complete either | |
| 14:46:31 | efried | aspiers: "want"... it's a suggestion I thought might interest you :) | |
| 14:46:37 | mriedem | sean-k-mooney: a few more small things in the pmu change https://review.opendev.org/#/c/671338/11 | |
| 14:46:48 | aspiers | efried: indeed, it will make me happy to see others using it :) | |
| 14:47:32 | sean-k-mooney | mriedem: ya i just got the email for your ps8 comments | |
| 14:47:36 | sean-k-mooney | ill look at them now | |
| 14:48:16 | sean-k-mooney | are you ok with me moveing the bool_form_strng as i have in the latest version? | |
| 14:49:21 | sean-k-mooney | assuming i add the new tests you suggested | |
| 14:50:35 | efried | mriedem: I think you were complaining about how long it takes to build specs at some point, and since you're clearly so enthusiastic about this kind of workflow improvement: https://review.opendev.org/#/c/673356/ | |
| 14:51:18 | mriedem | complain yes, but not so much to want to review a bash script :) | |
| 14:52:19 | sean-k-mooney | efried: the specs are not too bad. recently started building releasenotes locally for some of my patches and was suprised at how long those take. | |
| 14:52:21 | dansmith | mriedem: I dropped some comments on the bottom two patches, but I'm still mulling my feelings about it, but am out of time before my next call | |
| 14:53:07 | dansmith | mriedem: it really seems like an unfortunate purpose-built call to do part of the work of the other two for a specific situation, which just feels wrong and very fragile | |
| 14:53:09 | dansmith | I know it's more work to refactor, and I don't really want to review *that* either | |
| 14:53:16 | dansmith | so I dunno | |
| 14:57:15 | mriedem | to refactor prep_resize? | |
| 14:57:58 | dansmith | well, all of them to do this as a single thing | |
| 14:58:12 | dansmith | I dunno, I'll try to think on it when I'm done with my next call | |
| 14:58:14 | mriedem | i think that would be gross in other ways | |
| 14:58:18 | mriedem | ack | |
| 14:58:26 | dansmith | this just seems like piling on another copy of a bunch of existing logic | |
| 15:05:52 | frickler | mriedem: I'm cleaning up old patches and found https://bugs.launchpad.net/nova/+bug/1815082 , is that just forgotten or has it been resolved? | |
| 15:05:52 | openstack | Launchpad bug 1815082 in OpenStack Compute (nova) ""DBNonExistentTable: (sqlite3.OperationalError) no such table: services" when starting nova-metadata under uwsgi" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 15:06:23 | mriedem | it has not been resolved | |
| 15:07:00 | mriedem | dansmith: well the whole cross-cell series was also agreed to be separate from trying to munge it into existing compute manager code for sanity | |
| 15:07:18 | sean-k-mooney | efried: comments in https://review.opendev.org/#/c/673356/1 | |
| 15:07:24 | mriedem | i've refactored and made common the obvious stuff along the way like the notification bits | |
| 15:10:47 | mriedem | frickler: i ran out of steam trying to debug it | |
| 15:35:57 | mriedem | efried: i can +2 now if sean is ok with a FUP or +2 after you address his comments, either way https://review.opendev.org/#/c/673356/ | |
| 15:37:46 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: tox -e fast-specs https://review.opendev.org/673356 | |
| 15:38:20 | efried | sean-k-mooney, mriedem: Addressed some of the comments, but mostly for the sake of compromise. | |
| 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 | mriedem: I was trying to logstash that kombu thing. I should be able to search for | |
| 15:54:41 | efried | message:"TypeError: open: path should be string, bytes or os.PathLike, not _NormalAccessor" | |
| 15:54:41 | efried | right?? | |
| 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 | so i'm using: | |
| 16:01:47 | mriedem | message:"b'TypeError: open: path should be string, bytes or os.PathLike, not _NormalAccessor'" AND tags:"console" | |
| 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 | mriedem | and i have to hit the magnifying glass thing to re-query | |
| 16:03:07 | efried | weird, I'm getting lots of hits with that sea... oh, 'and' is case-sensitive | |
| 16:03:25 | efried | so yeah, I'm getting nothing. | |
| 16:03:34 | mriedem | then we need to talk to clarkb | |