Good Morning Everyone,
I would like to do two things:
- Update pagure in prod to 2.14.1 - Change the PAGURE_AUTH configuration key from 'fas' to 'openid' since legal just cleared the requirement for the FPCA on pagure.io https://lists.fedoraproject.org/archives/list/legal%40lists.fedoraproject.or...
For 2.14.1, while not released, the changelog is going to be: - Fix typo for walking the repo when creating a diff of a PR - Have the web-hook use the signed content and have a content-type header - Fix running the tests on jenkins via a couple of fixes to pagure-admin and skipping a couple of tests on jenkins due to the current pygit2/libgit2 situation in epel7
Thoughts on these?
Thanks, Pierre
+1 here.
If you could drop a short note here before you plan the work and then one when it's done that would be great, so people know to expect it.
kevin
On Wed, Mar 29, 2017 at 12:19:31PM +0200, Pierre-Yves Chibon wrote:
On Wed, Mar 29, 2017 at 12:13:08PM +0200, Patrick Uiterwijk wrote:
+1 on the update and config change.
Thanks, I will start on it right now then.
Log from IRC: 12:21:51 * | pingou upgrading pagure.io 12:24:12 pingou | upgrade done
So far so good :)
Happy hacking! Pierre
On Wed, Mar 29, 2017 at 12:25:07PM +0200, Pierre-Yves Chibon wrote:
On Wed, Mar 29, 2017 at 12:19:31PM +0200, Pierre-Yves Chibon wrote:
On Wed, Mar 29, 2017 at 12:13:08PM +0200, Patrick Uiterwijk wrote:
+1 on the update and config change.
Thanks, I will start on it right now then.
Log from IRC: 12:21:51 * | pingou upgrading pagure.io 12:24:12 pingou | upgrade done
So far so good :)
Unfortunately, wasn't so good. The new diffing algorithm seems to not behave as expected, so I downgraded to 2.13.2 which we were using. I'll see if I can tackle the issue and cut a 2.14.2 fixing this.
Sorry for any inconvenience, Pierre
On Tue, Mar 28, 2017 at 10:41 AM, Pierre-Yves Chibon pingou@pingoured.fr wrote:
Good Morning Everyone,
I would like to do two things:
- Update pagure in prod to 2.14.1
- Change the PAGURE_AUTH configuration key from 'fas' to 'openid' since
legal just cleared the requirement for the FPCA on pagure.io https://lists.fedoraproject.org/archives/list/legal% 40lists.fedoraproject.org/message/UWE4JIXEV76ZB54D33WEOSQGZR4Q25L5/
For 2.14.1, while not released, the changelog is going to be:
- Fix typo for walking the repo when creating a diff of a PR
- Have the web-hook use the signed content and have a content-type header
- Fix running the tests on jenkins via a couple of fixes to pagure-admin
and skipping a couple of tests on jenkins due to the current pygit2/libgit2 situation in epel7
Thoughts on these?
Thanks, Pierre
infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists. fedoraproject.org
Does this involve current FAS users to manually create an OpenID account or is there a method for bulk migrating current FAS accounts?
Or maybe Im the only noob without an OpenID? ;)
On Tue, Mar 28, 2017 at 11:11:59AM -0600, InvalidPath wrote:
On Tue, Mar 28, 2017 at 10:41 AM, Pierre-Yves Chibon pingou@pingoured.fr wrote:
Good Morning Everyone, I would like to do two things: - Update pagure in prod to 2.14.1 - Change the PAGURE_AUTH configuration key from 'fas' to 'openid' since legal  just cleared the requirement for the FPCA on pagure.io  https://lists.fedoraproject.org/archives/list/legal%40lists.fedoraproject.org/message/UWE4JIXEV76ZB54D33WEOSQGZR4Q25L5/
[..] Thanks for trimming the email your reply to
Does this involve current FAS users to manually create an OpenID account or is there a method for bulk migrating current FAS accounts? Or maybe Im the only noob without an OpenID? ;)
We are already using openid to log into pagure.io via FAS, this change in configuration will just drop the check for the FPCA. So if you have a FAS account, you have an OpenID account ;-)
Pierre
infrastructure@lists.fedoraproject.org
-
InvalidPath -
Kevin Fenzi -
Patrick Uiterwijk -
Pierre-Yves Chibon