*** brinzhang has joined #openstack-glance | 00:11 | |
*** rcernin has joined #openstack-glance | 01:59 | |
*** evrardjp has quit IRC | 04:37 | |
*** evrardjp has joined #openstack-glance | 04:37 | |
*** ratailor has joined #openstack-glance | 05:26 | |
*** udesale has joined #openstack-glance | 05:27 | |
*** tkajinam has quit IRC | 05:45 | |
*** tkajinam has joined #openstack-glance | 05:46 | |
*** brinzhang_ has joined #openstack-glance | 06:16 | |
*** brinzhang has quit IRC | 06:19 | |
openstackgerrit | Merged openstack/python-glanceclient master: Add Python3 victoria unit tests https://review.opendev.org/719084 | 07:59 |
---|---|---|
openstackgerrit | Merged openstack/glance_store master: Add Python3 victoria unit tests https://review.opendev.org/719083 | 08:17 |
*** brinzhang has joined #openstack-glance | 09:10 | |
*** brinzhang_ has quit IRC | 09:13 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/glance_store master: Imported Translations from Zanata https://review.opendev.org/718384 | 10:19 |
*** lpetrut has joined #openstack-glance | 10:30 | |
*** tkajinam has quit IRC | 10:31 | |
*** brinzhang_ has joined #openstack-glance | 11:18 | |
*** brinzhang has quit IRC | 11:21 | |
*** brinzhang has joined #openstack-glance | 11:24 | |
*** brinzhang_ has quit IRC | 11:27 | |
*** brinzhang_ has joined #openstack-glance | 11:27 | |
*** brinzhang has quit IRC | 11:28 | |
*** rosmaita has joined #openstack-glance | 11:39 | |
*** tkajinam has joined #openstack-glance | 12:05 | |
*** brinzhang has joined #openstack-glance | 12:15 | |
*** brinzhang_ has quit IRC | 12:18 | |
*** brinzhang_ has joined #openstack-glance | 12:36 | |
*** ratailor has quit IRC | 12:37 | |
*** udesale_ has joined #openstack-glance | 12:38 | |
*** brinzhang has quit IRC | 12:39 | |
*** udesale has quit IRC | 12:40 | |
*** jv has joined #openstack-glance | 13:01 | |
abhishekk | rosmaita, around? | 13:37 |
abhishekk | rosmaita, kindly have a look at below 2 patches, 1 is config refresh and 2nd is release notes | 13:39 |
abhishekk | https://review.opendev.org/719023 | 13:39 |
abhishekk | https://review.opendev.org/717679 | 13:39 |
rosmaita | abhishekk: ack | 13:39 |
abhishekk | rosmaita, thank you | 13:39 |
*** rcernin has quit IRC | 13:41 | |
openstackgerrit | Brian Rosmaita proposed openstack/glance master: Ussuri final release notes https://review.opendev.org/717679 | 13:59 |
rosmaita | abhishekk: fixed a formatting problem ^^ | 13:59 |
abhishekk | rosmaita, ack, thank you | 14:00 |
abhishekk | I will put +2 once zuul passes, kindly approve it | 14:00 |
rosmaita | will do | 14:00 |
abhishekk | so that I can submit the release patch | 14:00 |
abhishekk | thank you :D | 14:00 |
rosmaita | looks like we hit that sphinx change | 14:14 |
rosmaita | testing a change locally now | 14:16 |
abhishekk | rosmaita, locally fails with same error as well | 14:19 |
rosmaita | abhishekk: i'm getting an error about glance.hacking in autodoc | 14:20 |
rosmaita | are you seeing the neutron service not in glossary error? | 14:20 |
abhishekk | rosmaita, I am getting error, /opt/stack/glance/doc/source/glossary.rst:2539:term not in glossary: Networking Service (neutron) | 14:20 |
rosmaita | abhishekk: cool, change line 2539 so that the thing in <> | 14:21 |
rosmaita | says Newtorking service | 14:21 |
abhishekk | ack, let me test | 14:21 |
rosmaita | that will match line 2531 | 14:22 |
rosmaita | i think sphinx 3.0.0 or something made the heading-as-anchor case sensitive, used not to be | 14:22 |
smcginnis | rosmaita: Yep. I had to fix one of those with two anchors only being different by case. | 14:24 |
abhishekk | ohh, lets see, running it locally again | 14:24 |
smcginnis | Used to be fine, not anymore. | 14:25 |
rosmaita | guess that's why it was a major version change | 14:25 |
abhishekk | rosmaita, that error gone, now another error | 14:25 |
abhishekk | opt/stack/glance/doc/source/glossary.rst:2616:term not in glossary: Compute API (nova API) | 14:26 |
abhishekk | I guess same thing here as well | 14:26 |
rosmaita | smcginnis: on that topic, i was thinking we should run the docs job in the requirements gate to catch this kind of stuff ... was wondering if there's a way to run the docs jobs without publishing? | 14:26 |
rosmaita | abhishekk: yeah, sounds like it | 14:26 |
rosmaita | i wonder how many of those there will be | 14:27 |
abhishekk | :( | 14:27 |
rosmaita | may be worth doing a patch with the glossary changes only if you find > 3 | 14:27 |
smcginnis | rosmaita: I'm not sure if there is. And which ones would we run in the requirements gate? Some projects docs jobs are fine, others aren't. | 14:27 |
abhishekk | a lot of them | 14:28 |
smcginnis | I think this is a rare enough issue with this being a major version bump that we probably are better just handling when it happens. | 14:28 |
rosmaita | smcginnis: i vote for running the cinder and glance doc jobs :) | 14:28 |
smcginnis | I wouldn't expect it to happen too often, and probably not another major version bump for some time now. | 14:28 |
smcginnis | :) | 14:28 |
rosmaita | yeah, but these seem only to happen at milestone or RC time | 14:29 |
abhishekk | rosmaita, 3 changes | 14:31 |
rosmaita | up to you whether to hit them on this patch or another | 14:31 |
abhishekk | rosmaita, http://paste.openstack.org/show/792044/ | 14:32 |
abhishekk | We should go with this patch only, otherwise we need to wait until getting this patch merged | 14:32 |
rosmaita | works for me | 14:33 |
abhishekk | also gate is free today, so we could do it fast | 14:33 |
abhishekk | should I submit the change, or as latest patch is in your local, you will do it? | 14:33 |
rosmaita | i can do it | 14:34 |
abhishekk | cool, thank you | 14:34 |
*** lpetrut has quit IRC | 14:39 | |
openstackgerrit | Brian Rosmaita proposed openstack/glance master: Ussuri final release notes https://review.opendev.org/717679 | 14:40 |
rosmaita | let's see if that works | 14:41 |
abhishekk | ack | 14:41 |
abhishekk | it worked \o/ | 14:49 |
abhishekk | I will be back in 2 hours | 14:59 |
*** gyee has joined #openstack-glance | 15:39 | |
*** evrardjp has quit IRC | 16:37 | |
*** evrardjp has joined #openstack-glance | 16:37 | |
openstackgerrit | Merged openstack/glance master: Ussuri final release notes https://review.opendev.org/717679 | 17:01 |
*** udesale_ has quit IRC | 17:11 | |
*** jv has quit IRC | 19:03 | |
*** jv has joined #openstack-glance | 19:15 | |
abhishekk | rosmaita, still around? | 19:42 |
rosmaita | yes | 19:43 |
abhishekk | https://review.opendev.org/718551 | 19:43 |
rosmaita | looking | 19:45 |
rosmaita | yeah, that's kind of important | 19:45 |
abhishekk | ack | 19:46 |
abhishekk | yes | 19:46 |
openstackgerrit | Merged openstack/glance master: Refresh Glance example configs for Ussuri milestone 3 https://review.opendev.org/719023 | 19:46 |
abhishekk | smcginnis, you around? | 19:48 |
*** irclogbot_3 has quit IRC | 19:51 | |
*** irclogbot_3 has joined #openstack-glance | 19:52 | |
smcginnis | abhishekk: Kind of. What up? | 19:56 |
abhishekk | https://review.opendev.org/718551 | 19:56 |
abhishekk | Backport | 19:56 |
smcginnis | Oh right, that one. Looking! | 19:56 |
abhishekk | thank you to both of you, rosmaita and smcginnis | 19:57 |
rosmaita | np | 19:57 |
abhishekk | rosmaita, do I need to push reno patch for this bug fix before release? | 19:57 |
smcginnis | abhishekk: Just remember there is an extra step now that we are past the deadlines and into the requirements freeze: http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014060.html | 19:58 |
rosmaita | abhishekk: that would probably be a good idea so that it's clear why this release is happening | 19:58 |
rosmaita | but maybe smcginnis has an opinion about a release note? | 19:59 |
smcginnis | The other option being going out with it broken, then doing a bugfix release right after the freeze has been lifted post-coordinated release. | 19:59 |
smcginnis | abhishekk, rosmaita: Yeah, release note would probably be useful. | 19:59 |
abhishekk | rosmaita, ack, will prepare one patch soon | 19:59 |
abhishekk | smcginnis, ack | 19:59 |
rosmaita | ok, i will be around for a while | 20:00 |
abhishekk | smcginnis, so do I need to send FFE for this | 20:00 |
smcginnis | abhishekk: Yeah, if we want to get another client release out for ussuri, that will need a FFE approved. | 20:02 |
openstackgerrit | Merged openstack/python-glanceclient stable/ussuri: Pass --all-stores, --allow-failure as bool to API https://review.opendev.org/718551 | 20:02 |
abhishekk | smcginnis, ack | 20:04 |
abhishekk | smcginnis, http://lists.openstack.org/pipermail/openstack-discuss/2020-April/014090.html | 20:09 |
abhishekk | does it looks good | 20:09 |
smcginnis | Missed the [requirements] tag in there, but I'll mention it in #openstack-requirements to make sure it's seen. | 20:11 |
abhishekk | smcginnis, thank you | 20:12 |
abhishekk | smcginnis, will send another mail with [requirements] tag | 20:16 |
smcginnis | abhishekk: Oh, don't worry. | 20:20 |
smcginnis | Well, one of them will get an answer. :) | 20:21 |
abhishekk | :D | 20:24 |
abhishekk | smcginnis, last question | 20:24 |
abhishekk | I am getting below error while generating reno | 20:24 |
abhishekk | ERROR: InvocationError for command could not find executable reno | 20:25 |
smcginnis | Is that when running "tox -e releasenotes"? | 20:25 |
abhishekk | tox -e venv -- reno new slug-goes-here | 20:25 |
smcginnis | I had to fix something similar in cinder where the reno install was removed from the venv creation. | 20:26 |
smcginnis | You could do this instead: "tox -e releasenotes --notest; .tox/releasenotes/bin/reno new slug-goes-here" | 20:26 |
abhishekk | ohh | 20:27 |
abhishekk | thank you | 20:27 |
abhishekk | it worked | 20:27 |
openstackgerrit | Sean McGinnis proposed openstack/glance master: Install all deps in venv creation https://review.opendev.org/719680 | 20:35 |
smcginnis | abhishekk: That should take care of it so we can use reno like it's documented. ^ | 20:36 |
abhishekk | smcginnis, ack | 20:36 |
smcginnis | abhishekk: And FFE was approved, so we are all clear for doing another release as soon as we're ready. | 20:36 |
abhishekk | smcginnis great, thank you | 20:37 |
openstackgerrit | Abhishek Kekane proposed openstack/python-glanceclient stable/ussuri: Add release note for glanceclient 3.1.1 https://review.opendev.org/719681 | 20:40 |
abhishekk | smcginnis, rosmaita ^^^ | 20:41 |
rosmaita | looking | 20:41 |
abhishekk | I will tag glanceclient release today and glance m3 tomorrow (till the will get smcginnis latest patch https://review.opendev.org/719680 in as well) | 20:45 |
rosmaita | abhishekk: suggestion for a change noted on your patch | 20:45 |
abhishekk | rosmaita, ack | 20:45 |
openstackgerrit | Abhishek Kekane proposed openstack/python-glanceclient stable/ussuri: Add release note for glanceclient 3.1.1 https://review.opendev.org/719681 | 20:47 |
openstackgerrit | Abhishek Kekane proposed openstack/python-glanceclient stable/ussuri: Add release note for glanceclient 3.1.1 https://review.opendev.org/719681 | 20:48 |
abhishekk | rosmaita, done (sorry for shout out, I know you are busy with cinder as well) | 20:49 |
rosmaita | no worries | 20:49 |
abhishekk | smcginnis, if you around, https://review.opendev.org/719681 | 20:55 |
smcginnis | Just did it. ;) | 20:56 |
abhishekk | thanks :D | 20:56 |
abhishekk | Have a great day ahead both of you | 20:56 |
smcginnis | abhishekk: Have a good night. | 20:56 |
abhishekk | thanks | 20:56 |
openstackgerrit | Merged openstack/python-glanceclient stable/ussuri: Add release note for glanceclient 3.1.1 https://review.opendev.org/719681 | 21:00 |
abhishekk | signing out for the day | 21:24 |
rosmaita | have a good one (what's left of it!) | 21:24 |
abhishekk | thank you | 21:24 |
*** abhishekk is now known as abhishekk|away | 21:25 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!