openstackgerrit | Merged openstack/requirements stable/train: update constraint for os-brick to new release 2.10.2 https://review.opendev.org/714987 | 00:51 |
---|---|---|
openstackgerrit | Merged openstack/requirements stable/stein: update constraint for os-brick to new release 2.8.4 https://review.opendev.org/715022 | 00:51 |
*** tosky has quit IRC | 00:52 | |
*** udesale has joined #openstack-requirements | 04:47 | |
*** tonyb[m] has quit IRC | 04:49 | |
*** tonyb[m] has joined #openstack-requirements | 04:49 | |
*** udesale has quit IRC | 05:10 | |
*** udesale has joined #openstack-requirements | 05:12 | |
*** evrardjp has quit IRC | 05:36 | |
*** evrardjp has joined #openstack-requirements | 05:36 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/714839 | 06:18 |
openstackgerrit | Matthew Thode proposed openstack/requirements master: Updated from generate-constraints https://review.opendev.org/714839 | 06:36 |
*** ralonsoh has joined #openstack-requirements | 07:09 | |
*** vishalmanchanda has joined #openstack-requirements | 07:56 | |
*** evrardjp has quit IRC | 07:57 | |
*** evrardjp has joined #openstack-requirements | 08:01 | |
*** dmellado has quit IRC | 08:06 | |
*** dmellado has joined #openstack-requirements | 08:10 | |
*** rpittau|afk is now known as rpittau | 08:26 | |
*** toabctl has quit IRC | 08:30 | |
openstackgerrit | Simon Merrick proposed openstack/requirements stable/rocky: Bump pyScss upper constraint to 1.3.6 https://review.opendev.org/715133 | 08:39 |
*** e0ne has joined #openstack-requirements | 08:51 | |
*** tosky has joined #openstack-requirements | 09:00 | |
*** toabctl has joined #openstack-requirements | 09:14 | |
*** e0ne_ has joined #openstack-requirements | 09:25 | |
*** e0ne has quit IRC | 09:25 | |
*** e0ne_ has quit IRC | 09:50 | |
*** e0ne has joined #openstack-requirements | 09:51 | |
*** dtantsur|afk is now known as dtantsur | 10:48 | |
*** odyssey4me has joined #openstack-requirements | 10:53 | |
odyssey4me | hey folks, I've just raised https://bugs.launchpad.net/tripleo/+bug/1869174 - is it appropriate to add that pin in upper-constraints to cover all repositories? | 10:53 |
openstack | Launchpad bug 1869174 in tripleo "tripleo-common-stable/train openstack-tox-py27 job failing with ERROR: Package 'Pygments' requires a different Python: 2.7.17 not in '>=3.5'" [Undecided,New] | 10:53 |
odyssey4me | Pyments is a dependency being pulled in by Sphinx | 10:54 |
smcginnis | odyssey4me: Looks like we may want to cap that only in stable branches. | 10:54 |
smcginnis | https://github.com/pygments/pygments/releases/tag/2.6.1 | 10:54 |
smcginnis | Just dropped py2 support. On a bugfix version bump no less. | 10:55 |
*** e0ne has quit IRC | 10:59 | |
*** e0ne has joined #openstack-requirements | 10:59 | |
*** e0ne has quit IRC | 11:00 | |
*** e0ne has joined #openstack-requirements | 11:00 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements master: Add Pygments to upper-constraints https://review.opendev.org/715168 | 11:02 |
smcginnis | odyssey4me: Not against master. We would only want to cap on stable branches. | 11:03 |
smcginnis | Well, maybe we should on master. Let's see what prometheanfire thinks. | 11:04 |
smcginnis | odyssey4me: But there is a typo issue at least. | 11:04 |
odyssey4me | smcginnis: ok, should I push one up against the stable branches too then right now, or wait? | 11:04 |
smcginnis | odyssey4me: Let's wait a bit. | 11:05 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements master: Add Pygments to upper-constraints https://review.opendev.org/715168 | 11:05 |
odyssey4me | smcginnis: do we need to wait for that to merge before we pull it back to stable branches, or should I propose an equivalent patch to stable branches now? | 11:15 |
*** chandankumar has joined #openstack-requirements | 11:17 | |
smcginnis | odyssey4me: I think you can probably just propose it already. | 11:24 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/train: Add Pygments to upper-constraints https://review.opendev.org/715173 | 11:29 |
odyssey4me | smcginnis chandankumar ^ | 11:29 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/train: Add Pygments to upper-constraints https://review.opendev.org/715173 | 11:32 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/stein: Add Pygments to upper-constraints https://review.opendev.org/715175 | 11:34 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/rocky: Add Pygments to upper-constraints https://review.opendev.org/715177 | 11:36 |
*** rpittau is now known as rpittau|bbl | 11:45 | |
*** e0ne_ has joined #openstack-requirements | 11:46 | |
*** e0ne has quit IRC | 11:47 | |
*** e0ne has joined #openstack-requirements | 11:50 | |
*** e0ne_ has quit IRC | 11:50 | |
*** e0ne has quit IRC | 11:53 | |
*** e0ne has joined #openstack-requirements | 11:54 | |
*** e0ne has quit IRC | 12:14 | |
*** e0ne has joined #openstack-requirements | 12:14 | |
*** udesale_ has joined #openstack-requirements | 12:21 | |
*** udesale has quit IRC | 12:24 | |
smcginnis | prometheanfire: Any background on why pygments is in the blacklist? | 12:25 |
smcginnis | https://opendev.org/openstack/requirements/src/branch/master/blacklist.txt#L18 | 12:25 |
smcginnis | It's not a linter. | 12:25 |
smcginnis | Hmm, https://opendev.org/openstack/requirements/commit/aee6e953804da859ecf55e16c6a6cf5ac7571e6b | 12:26 |
*** e0ne has quit IRC | 12:28 | |
*** e0ne has joined #openstack-requirements | 12:28 | |
*** elod has joined #openstack-requirements | 12:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for python-openstackclient to new release 5.2.0 https://review.opendev.org/715195 | 12:46 |
odyssey4me | smcginnis: oh dear: https://opendev.org/openstack/requirements/commit/aee6e953804da859ecf55e16c6a6cf5ac7571e6b | 13:02 |
odyssey4me | lol, jinx | 13:02 |
smcginnis | odyssey4me: We need to remove it from the blacklist.txt file. The patch that put it there isn't really clear why it was put there in the first place though. | 13:06 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements master: Add Pygments to upper-constraints https://review.opendev.org/715168 | 13:06 |
smcginnis | It's used for docs and other things - definitely not a linter - so I don't think it was correct to lump it in there. | 13:06 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements master: Add Pygments to upper-constraints https://review.opendev.org/715168 | 13:08 |
odyssey4me | smcginnis: ok, I've included a reference to that change-id with an explanation... so I think we have a complete reference now | 13:08 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/requirements master: update constraint for openstacksdk to new release 0.44.0 https://review.opendev.org/715199 | 13:10 |
*** rpittau|bbl is now known as rpittau | 13:10 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/train: Add Pygments to upper-constraints https://review.opendev.org/715173 | 13:10 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/stein: Add Pygments to upper-constraints https://review.opendev.org/715175 | 13:11 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/requirements stable/rocky: Add Pygments to upper-constraints https://review.opendev.org/715177 | 13:12 |
*** e0ne_ has joined #openstack-requirements | 13:41 | |
*** e0ne has quit IRC | 13:41 | |
odyssey4me | smcginnis: apologies, I see you +2'd just before I updated the commit msg in https://review.opendev.org/715168 | 13:44 |
smcginnis | No worries. | 13:45 |
prometheanfire | odyssey4me: sup yo | 14:05 |
smcginnis | prometheanfire: Any recollection of why we would have pygments in blacklist when it's not a linter and is used by many projects? | 14:06 |
prometheanfire | smcginnis: nope | 14:06 |
prometheanfire | it's mentioned that it's not a runtime dep, but that's not useful | 14:07 |
smcginnis | The patch that put it there wasn't at all clear. We need to have it in u-c so new releases don't break stable. | 14:07 |
smcginnis | Mostly an issue for the branches with py2 support now that they've dropped it. | 14:07 |
smcginnis | But seems like something we should track regardless. | 14:07 |
prometheanfire | yep | 14:07 |
odyssey4me | prometheanfire: o/ | 14:23 |
*** chandan_kumar has joined #openstack-requirements | 14:29 | |
*** chandankumar has quit IRC | 14:29 | |
*** dulek has quit IRC | 14:30 | |
*** dulek has joined #openstack-requirements | 14:33 | |
*** KeithMnemonic has quit IRC | 15:13 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/requirements master: Update upper-constraints for pyScss to 1.3.7 https://review.opendev.org/715240 | 15:14 |
odyssey4me | prometheanfire: are we a go? zuul says yes: https://review.opendev.org/715168 | 15:16 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/requirements master: Update upper-constraints for pyScss to 1.3.7 https://review.opendev.org/715240 | 15:16 |
prometheanfire | openstackgerrit: yep, voted (was waiting for zuul) | 15:20 |
prometheanfire | odyssey4me: heh ^ | 15:22 |
odyssey4me | prometheanfire: :) | 15:23 |
*** udesale_ has quit IRC | 15:50 | |
*** chandan_kumar is now known as chandankumar | 15:51 | |
*** number80 has quit IRC | 16:16 | |
*** number80 has joined #openstack-requirements | 16:22 | |
*** e0ne_ has quit IRC | 17:12 | |
*** rpittau is now known as rpittau|afk | 17:12 | |
*** e0ne has joined #openstack-requirements | 17:12 | |
*** ccamacho has joined #openstack-requirements | 17:31 | |
*** dtantsur is now known as dtantsur|afk | 17:32 | |
*** ccamacho has quit IRC | 17:32 | |
*** evrardjp has quit IRC | 17:49 | |
*** evrardjp has joined #openstack-requirements | 17:49 | |
*** openstack has quit IRC | 17:49 | |
*** openstack has joined #openstack-requirements | 17:51 | |
*** ChanServ sets mode: +o openstack | 17:51 | |
*** ralonsoh has quit IRC | 18:51 | |
*** e0ne has joined #openstack-requirements | 19:18 | |
*** e0ne has quit IRC | 19:39 | |
tbarron | people were just talking about pygments and stable branches and py2? | 19:50 |
tbarron | known issue that pygments 2.6+ breaks python2 jobs on stable branches? | 19:54 |
openstackgerrit | Harald Jensås proposed openstack/requirements stable/train: Cap Pygments <2.6.0 for Python 2.7 https://review.opendev.org/715300 | 20:01 |
smcginnis | tbarron: Yep, there's a patch making its way through. | 20:07 |
tbarron | smcginnis: thanks | 20:07 |
*** e0ne has joined #openstack-requirements | 20:20 | |
*** e0ne has quit IRC | 20:29 | |
*** e0ne has joined #openstack-requirements | 20:29 | |
*** e0ne has quit IRC | 20:42 | |
openstackgerrit | Merged openstack/requirements master: Updated from generate-constraints https://review.opendev.org/714839 | 21:46 |
*** prometheanfire has quit IRC | 23:04 | |
*** vishalmanchanda has quit IRC | 23:39 | |
*** tosky has quit IRC | 23:53 | |
openstackgerrit | Merged openstack/requirements master: Add Pygments to upper-constraints https://review.opendev.org/715168 | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!