| *** yanyanhu has joined #openstack-release | 01:35 | |
| *** amotoki has joined #openstack-release | 02:57 | |
| *** amotoki has quit IRC | 02:59 | |
| *** amotoki has joined #openstack-release | 02:59 | |
| *** Jeffrey4l has quit IRC | 03:05 | |
| *** Jeffrey4l has joined #openstack-release | 03:05 | |
| *** mwhahaha has quit IRC | 03:26 | |
| *** mwhahaha has joined #openstack-release | 03:29 | |
| *** sdake has joined #openstack-release | 03:56 | |
| *** sdake has quit IRC | 04:28 | |
| *** Jeffrey4l has quit IRC | 04:53 | |
| *** hongbin has joined #openstack-release | 04:53 | |
| *** hongbin has quit IRC | 05:11 | |
| *** amotoki has quit IRC | 05:32 | |
| *** amotoki has joined #openstack-release | 05:55 | |
| *** amotoki has quit IRC | 05:58 | |
| *** amotoki has joined #openstack-release | 06:18 | |
| *** oanson has joined #openstack-release | 06:59 | |
| *** amotoki has quit IRC | 07:10 | |
| *** amotoki has joined #openstack-release | 07:32 | |
| *** pcaruana has joined #openstack-release | 07:33 | |
| *** claudiub has joined #openstack-release | 07:59 | |
| *** shardy has joined #openstack-release | 08:02 | |
| *** ihrachys has joined #openstack-release | 08:04 | |
| *** amotoki has quit IRC | 08:06 | |
| *** amotoki has joined #openstack-release | 08:25 | |
| *** amotoki has quit IRC | 08:40 | |
| *** jpich has joined #openstack-release | 08:50 | |
| *** jtomasek has joined #openstack-release | 08:57 | |
| *** jtomasek has quit IRC | 09:19 | |
| *** lucas-afk is now known as lucasagomes | 09:26 | |
| *** yanyanhu has quit IRC | 09:39 | |
| *** amotoki has joined #openstack-release | 09:56 | |
| *** amotoki has quit IRC | 10:06 | |
| *** amotoki has joined #openstack-release | 10:09 | |
| *** electrofelix has joined #openstack-release | 10:11 | |
| *** amotoki has quit IRC | 10:23 | |
| *** Jeffrey4l has joined #openstack-release | 10:24 | |
| *** amotoki has joined #openstack-release | 10:38 | |
| *** apuimedo has quit IRC | 10:41 | |
| *** amotoki has quit IRC | 10:45 | |
| *** ihrachys has quit IRC | 11:12 | |
| *** dave-mccowan has joined #openstack-release | 11:26 | |
| *** esmiurium has joined #openstack-release | 11:30 | |
| *** sdague has joined #openstack-release | 11:33 | |
| *** wolverineav has joined #openstack-release | 11:42 | |
| *** ihrachys has joined #openstack-release | 11:43 | |
| *** dave-mcc_ has joined #openstack-release | 11:45 | |
| *** wolverineav has quit IRC | 11:47 | |
| *** wolverineav has joined #openstack-release | 11:47 | |
| *** dave-mccowan has quit IRC | 11:48 | |
| *** wolverin_ has joined #openstack-release | 11:48 | |
| *** wolverineav has quit IRC | 11:52 | |
| *** lucasagomes is now known as lucas-hungry | 11:55 | |
| dims | dhellmann : ttx : o/ | 11:56 |
|---|---|---|
| openstackgerrit | Merged openstack/releases: newton: release nova 14.0.3 https://review.openstack.org/412092 | 11:59 |
| openstackgerrit | Merged openstack/releases: Release octavia 0.9.1 https://review.openstack.org/412026 | 12:01 |
| openstackgerrit | Merged openstack/releases: Release octavia 0.8.1 https://review.openstack.org/412025 | 12:01 |
| openstackgerrit | Merged openstack/releases: H803 hacking have been deprecated https://review.openstack.org/412186 | 12:01 |
| openstackgerrit | Merged openstack/releases: teach missing-releases to look for artifacts as well as tags https://review.openstack.org/411530 | 12:04 |
| openstackgerrit | Merged openstack/releases: add validation rule to check tarball-base https://review.openstack.org/411989 | 12:04 |
| openstackgerrit | Merged openstack/releases: fix tarball-base values for newton and ocata https://review.openstack.org/411977 | 12:04 |
| openstackgerrit | Merged openstack/releases: Release oslo.db 4.13.5 (newton) https://review.openstack.org/411314 | 12:04 |
| openstackgerrit | Merged openstack/releases: Release searchlight-ui 1.0.1 (newton stable) https://review.openstack.org/411492 | 12:04 |
| *** Jeffrey4l has quit IRC | 12:10 | |
| *** shardy is now known as shardy_lunch | 12:15 | |
| *** dave-mcc_ has quit IRC | 12:18 | |
| *** sdake has joined #openstack-release | 12:18 | |
| *** askb has quit IRC | 12:19 | |
| *** sdake_ has joined #openstack-release | 12:22 | |
| *** sdake has quit IRC | 12:23 | |
| *** rtheis has joined #openstack-release | 12:25 | |
| *** sdake has joined #openstack-release | 12:29 | |
| *** sdake_ has quit IRC | 12:32 | |
| *** dulek has joined #openstack-release | 12:41 | |
| dulek | dhellmann: Hi, can you give me a pointer about how to fix such release notes fail: http://docs-draft.openstack.org/00/410200/3/check/gate-cinder-releasenotes/ec130d5//releasenotes/build/html/unreleased.html ? | 12:42 |
| *** dulek has left #openstack-release | 12:42 | |
| *** dulek has joined #openstack-release | 12:42 | |
| dulek | dhellmann: I've tried modifying the release note itself, but it keeps generating it the same way. | 12:43 |
| *** Jeffrey4l has joined #openstack-release | 12:43 | |
| *** flaper87 has joined #openstack-release | 12:45 | |
| openstackgerrit | Merged openstack/releases: add a verbose flag to list-deliverables https://review.openstack.org/411438 | 12:53 |
| openstackgerrit | Merged openstack/releases: add --missing-milestone flag to list-deliverables https://review.openstack.org/411439 | 12:53 |
| dims | dulek : missing a "-" in front of line 3? http://git.openstack.org/cgit/openstack/cinder/tree/releasenotes/notes/vmax-clone-cg-09fce492931c957f.yaml | 12:54 |
| dims | dulek : compare to http://git.openstack.org/cgit/openstack/cinder/tree/releasenotes/notes/vmax-attach-snapshot-3137e59ab4ff39a4.yaml | 12:54 |
| dulek | dims: Yeah, I've noticed that. The problem is - I'm fixing this, *committing* the change, and "tox -e releasenotes" builds same thing. | 12:56 |
| *** jtomasek has joined #openstack-release | 12:56 | |
| dulek | dims: While "reno report ." generates it correctly… | 12:56 |
| dims | dulek : i'd just make sure that a change with '-' gets merged. and check what happens in the *next* run after that | 12:57 |
| dulek | dims: Hm, okay, why not. | 12:57 |
| dulek | dims: Thanks. | 12:57 |
| dims | dulek : cross my fingers :) | 12:59 |
| *** gordc has joined #openstack-release | 12:59 | |
| *** lucas-hungry is now known as lucasagomes | 13:01 | |
| ttx | dims: o/ | 13:03 |
| *** trown|outtypewww is now known as trown | 13:14 | |
| openstackgerrit | Witold Bedyk proposed openstack/releases: Release Monasca https://review.openstack.org/411842 | 13:28 |
| *** weshay_lata is now known as weshay | 13:31 | |
| *** tommylikehu_ has joined #openstack-release | 13:38 | |
| *** hongbin has joined #openstack-release | 14:05 | |
| *** bobh has joined #openstack-release | 14:09 | |
| *** shardy_lunch is now known as shardy | 14:16 | |
| *** sdake_ has joined #openstack-release | 14:21 | |
| andymccr | morning releases team - im running into an issue since the validation check was added for releases gate: https://review.openstack.org/#/c/411196/ - it seems we have a stale reference to an old repo name, which I'm fixing now, but since it's checking existing releases/tags - they will still fail that gate (i.e. i'm not sure how i could get a release to pass since I can't backport the fix to a tag) | 14:21 |
| *** sdague_ has joined #openstack-release | 14:21 | |
| *** sdake has quit IRC | 14:23 | |
| *** sdague has quit IRC | 14:24 | |
| openstackgerrit | gordon chung proposed openstack/releases: oslo.messaging 5.16.0 (ocata) https://review.openstack.org/412471 | 14:26 |
| *** sdague_ is now known as sdague | 14:27 | |
| *** bobh has quit IRC | 14:41 | |
| dhellmann | o/ | 14:45 |
| dhellmann | dims, dulek : it's possible that sphinx is caching some information about the release notes page. You may have to remove releasenotes/build before trying again. | 14:45 |
| dhellmann | andymccr : I'll take a look after my morning meetings and try to get it fixed today | 14:48 |
| andymccr | dhellmann: thanks again for the help, I'll be working on getting the fix into the repository so that at least going forward its correct. | 14:48 |
| dhellmann | andymccr : wait, is this just the tarball base thing? | 14:49 |
| dhellmann | that should be easy to fix? | 14:49 |
| *** jtomasek has quit IRC | 14:50 | |
| andymccr | dhellmann: the tarball validate, yes i thinkso - i think the issue is it runs setup.py in the repo which returns a sdist value taht doesnt match | 14:50 |
| andymccr | thats easy to fix | 14:50 |
| dhellmann | yes, don't change your repo, change the releases repo | 14:50 |
| dhellmann | I thought I caught all of those, hang on | 14:51 |
| andymccr | dhellmann: ahh ok, my understanding was there is a mismatch between the repo name and the setup.cfg in the repo (which there is) | 14:51 |
| dhellmann | right, there's a field you can set in the releases repo to tell us what the name really is | 14:51 |
| dhellmann | the logic to build the releases.o.o site guesses based on the repo name, because checking out every repo to interrogate it is too expensive | 14:51 |
| dhellmann | and most do match the repo name | 14:52 |
| andymccr | oh great - i mean i'll fix it in the repo itself too, because going forward we shouldnt need to specify, but for existing releases i can then specify the value | 14:52 |
| dhellmann | I think if you rebase your patch on current master it should pass; I fixed that issue as part of adding the validation logic | 14:52 |
| andymccr | oh nice | 14:52 |
| andymccr | ok will give that a go, thanks! | 14:52 |
| dhellmann | oh, well, if you change it in the repo that's going to break the validation again | 14:52 |
| andymccr | oh | 14:52 |
| dulek | dhellmann: I've actually did that. And we've fixed that in tree without effect: https://review.openstack.org/#/c/412445/ | 14:52 |
| openstackgerrit | Andy McCrae proposed openstack/releases: Release OpenStack-Ansible Newton/14.0.4 https://review.openstack.org/411196 | 14:53 |
| andymccr | ive blocked the patch i filed against the repository so long | 14:53 |
| dhellmann | dulek, smcginnis : when I download 412445 and build the release notes I see no error. I can't tell from your last comment if you're still seeing an error or not. | 14:57 |
| smcginnis | dhellmann: I do get the problem when I run that locally. | 14:57 |
| dhellmann | smcginnis : you've removed releasenotes/build and committed the changes to the note file? | 14:57 |
| smcginnis | dhellmann: To be clear, it doesn't give and error. But that release note gets rendered one character per line. | 14:57 |
| dhellmann | oh, that's not what I was looking for | 14:58 |
| dhellmann | stand by then | 14:58 |
| smcginnis | dhellmann: Fresh repo clone, updated the release note and committed it, then built. | 14:58 |
| *** jtomasek has joined #openstack-release | 14:59 | |
| smcginnis | dhellmann: Line 80 in unreleased.html, at least for mine. | 14:59 |
| *** tommylikehu_ has quit IRC | 15:01 | |
| dhellmann | ok, I can reproduce that | 15:02 |
| *** bobh has joined #openstack-release | 15:06 | |
| *** hongbin has quit IRC | 15:06 | |
| *** bobh has quit IRC | 15:07 | |
| *** bobh has joined #openstack-release | 15:07 | |
| *** bobh has quit IRC | 15:12 | |
| *** jtomasek has quit IRC | 15:13 | |
| dhellmann | smcginnis : https://review.openstack.org/412493 fixes the rendering issue for patches under review; you need to backport that to all stable branches to ensure you're testing notes properly. | 15:19 |
| openstackgerrit | Andy McCrae proposed openstack/releases: Fix existing tarball-base for openstack-ansible-os_ironic https://review.openstack.org/412498 | 15:30 |
| *** tommylikehu_ has joined #openstack-release | 15:30 | |
| dhellmann | andymccr : ^^ looks good; you'll likely have to rebase the release patch on top of that | 15:32 |
| andymccr | dhellmann: will do. thanks for the pointers - i'll need to fix the new release to add that tarball-base too I think! | 15:33 |
| dhellmann | andymccr : yes, that's true | 15:33 |
| dhellmann | andymccr : do you use one of our tools to automatically add new releases or do you do that by hand? | 15:33 |
| andymccr | dhellmann: we have a few scripts built into our main repository to do it automatically, but i'm going to fix that issue moving forward so further releases will be as normal | 15:34 |
| dhellmann | we should probably update the tool to add those values if they're needed | 15:34 |
| andymccr | won't impact existing releases | 15:34 |
| dhellmann | ok, we have a tool in the releases repo, too, so you may want to look at whether the tooling can be shared | 15:34 |
| dhellmann | we actually have 2 tools | 15:34 |
| dhellmann | because why not? | 15:34 |
| andymccr | dhellmann: haha excellent, i'll take a look - i know ours became painful because we have so many seperate repositories, but i think we should be able to use one of the existing ones | 15:35 |
| dhellmann | andymccr : by "fix that issue" I hope you don't mean changing the sdist name in the repos and breaking all of the *old* links to deliverables | 15:35 |
| andymccr | dhellmann: i believe it won't! It'd just change references to the old repo name (i believe the setup.cfg one is causing the issues) - that way existing releases won't be impacted since the cahnge can only be made to head of the branches and not the tagged releases | 15:36 |
| dhellmann | andymccr : when we build releases.o.o and build links to the existing artifact files, we do not go look back at the history of your repos to see what the name was at the time | 15:37 |
| dhellmann | so please don't change the sdist name in your repos, and just add the right tarball-base values to the deliverable files in the releases repo | 15:37 |
| dhellmann | basically, once you start building a deliverable, you can't change its sdist name. If you want to do that, you have to change the deliverable itself | 15:37 |
| dhellmann | which is fine, because changing the sdist name produces a new deliverable on pypi or wherever else the artifacts are published | 15:38 |
| dhellmann | does that make sense? | 15:39 |
| andymccr | dhellmann: ahh ok i think that makes sense, the only changes I want to make are to remove references inside the repo (e.g. in the setup.cfg name value) - is that the one that would break it? | 15:40 |
| *** weshay is now known as weshay_afk | 15:41 | |
| dhellmann | andymccr : changing the name value in setup.cfg would result in a new sdist filename, so yes, that would break the links to old deliverables (and if you uploaded to pypi it would break that) | 15:41 |
| andymccr | ok so the sdist name is created off head of the branch rather than the tag number - so that is to say if i changed it now, it would break existing builds because the that command isn't run on each deliverable? | 15:42 |
| dhellmann | there are a couple of different times we need the name, and not all of them are smart enough to know about changes in time | 15:43 |
| dhellmann | if you change the name in master, new builds in master would use that name | 15:43 |
| dhellmann | that would be fine if you were only interested in those packages | 15:43 |
| dhellmann | the releases website build looks at the repository name (not its contents) to get the name | 15:44 |
| smcginnis | dhellmann: Thanks for the fix! So to backport, that should be "branch: origin/master" for all of them? Or does it need to be per stable branch? | 15:44 |
| dhellmann | smcginnis : you need to *remove* the branch specifier completely in unreleased.rst | 15:44 |
| andymccr | dhellmann: ahh excellent ok so perhaps the best is to just change it in master and continue adding tarball-base for older releases - does that make sense as an approach? Ideally I'd like to avoid having to carry tarball-base for the rest of time (not that its a huge thing) | 15:44 |
| smcginnis | dhellmann: Oh, I looked too quick! Got it. :) | 15:44 |
| dhellmann | andymccr : I'm having trouble explaining just how terrible it would be for you to change the names of these things. EVERYTHING would break. Why do you want to change the names? | 15:45 |
| *** weshay_afk is now known as weshay | 15:45 | |
| andymccr | dhellmann: haha mainly just because it seems odd to have to specify a tarball-base because of a ref that doesnt match the reponame, with no way to adjust it moving forward - but i think the reality is it's not that big a deal, and clearly the pros outway the cons :) | 15:47 |
| andymccr | thanks for the explanation though, i have a better understanding of how that hangs together which is pretty useful | 15:47 |
| dhellmann | andymccr : we could change the validation logic to only apply it to new releases at some point, but we're trying to ensure that we have accurate values for all current releases. | 15:48 |
| andymccr | dhellmann: yeah that does make sense. | 15:48 |
| dhellmann | but that ignores the point that if you change the name of the thing you ship, your users need to know to use the new name instead of the old, so you're actually shipping something different than before | 15:48 |
| dhellmann | in your case, I imagine your users use the git repo directly, but in most cases they're looking for a file on tarballs.o.o or on pypi.python.org | 15:49 |
| andymccr | yeah - i think the thing is we don't ship anything. there is a process by which the roles are cloned based on what roles you're using - so its not really a shipped tarball. | 15:49 |
| dhellmann | and when those names change, that's a much bigger deal | 15:49 |
| dhellmann | right, but we haven't built the system to allow for your special case | 15:49 |
| andymccr | but i get that in this case perhaps its different, but still creates an issue on a larger scale. so im happy to just keep it as is going forward | 15:49 |
| dhellmann | k | 15:49 |
| andymccr | thanks :) | 15:49 |
| dhellmann | :-) | 15:49 |
| openstackgerrit | Andy McCrae proposed openstack/releases: Release OpenStack-Ansible Newton/14.0.4 https://review.openstack.org/411196 | 15:51 |
| *** sdague has quit IRC | 16:06 | |
| *** hongbin has joined #openstack-release | 16:10 | |
| *** jtomasek has joined #openstack-release | 16:12 | |
| *** pcaruana has quit IRC | 16:15 | |
| *** jtomasek has quit IRC | 16:16 | |
| *** cody-somerville has joined #openstack-release | 16:17 | |
| *** cody-somerville has quit IRC | 16:17 | |
| *** cody-somerville has joined #openstack-release | 16:17 | |
| *** jtomasek has joined #openstack-release | 16:22 | |
| *** TravT has joined #openstack-release | 16:26 | |
| *** TravT has quit IRC | 16:33 | |
| *** weshay is now known as weshay_bbiab | 16:34 | |
| *** jtomasek has quit IRC | 16:35 | |
| openstackgerrit | Eric Harney proposed openstack/releases: Cinder stable/newton 9.1.1 release https://review.openstack.org/412533 | 17:00 |
| *** jpich has quit IRC | 17:04 | |
| *** sdake_ is now known as sdake | 17:15 | |
| *** SergeyLukjanov__ has quit IRC | 17:34 | |
| *** SergeyLukjanov has joined #openstack-release | 17:35 | |
| *** trown is now known as trown|lunch | 17:48 | |
| *** sdague has joined #openstack-release | 17:48 | |
| *** lucasagomes is now known as lucas-afk | 17:50 | |
| *** weshay_bbiab is now known as weshay | 17:51 | |
| *** Jeffrey4l has quit IRC | 18:07 | |
| *** shardy has quit IRC | 18:08 | |
| *** armax has joined #openstack-release | 18:25 | |
| openstackgerrit | Merged openstack/releases: Fix existing tarball-base for openstack-ansible-os_ironic https://review.openstack.org/412498 | 18:33 |
| *** csomerville has joined #openstack-release | 18:36 | |
| *** ihrachys has quit IRC | 18:37 | |
| *** cody-somerville has quit IRC | 18:39 | |
| openstackgerrit | Merged openstack/releases: oslo.messaging 5.16.0 (ocata) https://review.openstack.org/412471 | 18:41 |
| openstackgerrit | Merged openstack/releases: Release Monasca https://review.openstack.org/411842 | 18:45 |
| *** cody-somerville has joined #openstack-release | 18:56 | |
| *** csomerville has quit IRC | 18:58 | |
| *** electrofelix has quit IRC | 18:59 | |
| *** david-lyle_ has joined #openstack-release | 19:13 | |
| *** openstackstatus has quit IRC | 19:13 | |
| *** david-lyle has quit IRC | 19:13 | |
| *** openstack has joined #openstack-release | 19:14 | |
| *** armax has quit IRC | 19:21 | |
| *** trown|lunch is now known as trown | 19:25 | |
| *** armax has joined #openstack-release | 19:31 | |
| dims | fungi : i was looking at versions advertised by the packages we list in global requirements - http://pastebin.com/NLKFZdS5 - a few of our own packages don't list 3.5 support | 19:48 |
| fungi | dims: this does not surprise me in the least | 19:49 |
| fungi | dims: one of the items called out on the cross-project spec is to make sure trove classifiers are correctly updated (in setup.cfg or possibly setup.py) | 19:50 |
| dims | right. +1 | 19:50 |
| amrith | did someone say trove? | 20:03 |
| *** breton_ is now known as breton | 20:04 | |
| sigmavirus | amrith: not openstack trove | 20:04 |
| sigmavirus | ;) | 20:04 |
| sigmavirus | amrith: trove classifiers are things in Python packaging (https://pypi.python.org/pypi?:action=list_classifiers) | 20:05 |
| amrith | what, something that I didn't screw up; Woo Hoo! this is a first | 20:05 |
| amrith | sigmavirus, thanks! | 20:05 |
| sigmavirus | Happy to help, amrith :) | 20:05 |
| amrith | thx sigmavirus .. we do often get bugs about some sword not working if you have a knife and a headache and stuff like that; apparently there's some roleplaying game called trove :) | 20:06 |
| sigmavirus | lol | 20:06 |
| sigmavirus | there's a video game called rust | 20:07 |
| sigmavirus | so I always wonder if the rust-lang team gets grief over that | 20:07 |
| amrith | https://www.trovegame.com/en/play-now/ | 20:08 |
| sigmavirus | naming things is hard, amrith | 20:10 |
| clarkb | what about Go | 20:14 |
| clarkb | iirc there was another languag with the same name and google said too bad we win because we are google | 20:14 |
| openstackgerrit | Doug Hellmann proposed openstack/releases: only check the tarball-base when building tarballs https://review.openstack.org/412612 | 20:39 |
| *** weshay is now known as weshay_relocatin | 20:42 | |
| *** askb has joined #openstack-release | 20:45 | |
| *** rockyg has joined #openstack-release | 21:02 | |
| *** v12aml has quit IRC | 21:08 | |
| *** v12aml has joined #openstack-release | 21:11 | |
| *** rtheis has quit IRC | 21:13 | |
| *** l4yerffeJ_ has joined #openstack-release | 21:33 | |
| *** l4yerffeJ has quit IRC | 21:35 | |
| *** bobh has joined #openstack-release | 21:56 | |
| *** bobh has quit IRC | 21:58 | |
| *** trown is now known as trown|outtypewww | 21:59 | |
| *** dave-mccowan has joined #openstack-release | 22:02 | |
| *** dave-mccowan has quit IRC | 22:07 | |
| *** dave-mccowan has joined #openstack-release | 22:11 | |
| *** wolverin_ has quit IRC | 22:41 | |
| *** wolverineav has joined #openstack-release | 22:50 | |
| *** wolverineav has quit IRC | 22:55 | |
| *** sdake has quit IRC | 23:02 | |
| *** cody-somerville has quit IRC | 23:03 | |
| *** rockyg has quit IRC | 23:05 | |
| openstackgerrit | Emilien Macchi proposed openstack/releases: Release TripleO Newton-2 https://review.openstack.org/412643 | 23:09 |
| *** wolverineav has joined #openstack-release | 23:11 | |
| openstackgerrit | Emilien Macchi proposed openstack/releases: Release TripleO Newton-2 https://review.openstack.org/412643 | 23:11 |
| *** wolverineav has quit IRC | 23:15 | |
| *** tommylikehu_ has quit IRC | 23:22 | |
| *** wolverineav has joined #openstack-release | 23:31 | |
| *** wolverineav has quit IRC | 23:35 | |
| *** gordc has quit IRC | 23:37 | |
| *** tommylikehu_ has joined #openstack-release | 23:37 | |
| *** sdague has quit IRC | 23:44 | |
| *** dave-mccowan has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!