Re: [PATCH] Make sure git and git-email are installed on puppet systems
On 2009-03-12 02:20:57 AM, Jesse Keating wrote: --- manifests/servergroups/puppet.pp |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/manifests/servergroups/puppet.pp b/manifests/servergroups/puppet.pp index d054fef..c393f9a 100644 --- a/manifests/servergroups/puppet.pp +++ b/manifests/servergroups/puppet.pp @@ -13,6 +13,8 @@ class puppetServer { include scripts::geoip-retriever include geoip-retriever include drbackupPubKey +include git-package +include git-email-package # Firewall Rules, allow web, smolt, Plone, mirrormanager, noc, pkgdb, certmaster and bodhi traffic through $tcpPorts = [ 80, 8140, 873, 51235 ] -- 1.5.5.6 +1 Thanks, Ricky pgp3oAr1vJGJv.pgp Description: PGP signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: [PATCH] Make sure git and git-email are installed on puppet systems
On Thu, 12 Mar 2009, Ricky Zhou wrote: On 2009-03-12 02:20:57 AM, Jesse Keating wrote: --- manifests/servergroups/puppet.pp |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/manifests/servergroups/puppet.pp b/manifests/servergroups/puppet.pp index d054fef..c393f9a 100644 --- a/manifests/servergroups/puppet.pp +++ b/manifests/servergroups/puppet.pp @@ -13,6 +13,8 @@ class puppetServer { include scripts::geoip-retriever include geoip-retriever include drbackupPubKey +include git-package +include git-email-package # Firewall Rules, allow web, smolt, Plone, mirrormanager, noc, pkgdb, certmaster and bodhi traffic through $tcpPorts = [ 80, 8140, 873, 51235 ] -- 1.5.5.6 +1 +1 -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change request - Using git send-email
On Thursday, March 12 2009, Jesse Keating said: Here is a message from git send-email. To get here, I used: $ git commit -a Created commit cb85f54: Disable rawhide. 1 files changed, 2 insertions(+), 1 deletions(-) $ git format-patch HEAD^ 0001-Disable-rawhide.patch $ git send-email --compose --to Fedora-infrastructure-list@redhat.com 0001-Disable-rawhide.patch Note that if it's a single patch, --compose is probably overkill as it makes two messages rather than one. Your commit message is self-explanatory, no? (... and if not, edit the 0001 file before running git send-email) Jeremy ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Change request -- fas template csrf fix
Found a template in fas that is not adding the csrf token properly. The Add User button on: https://admin.fedoraproject.org/accounts/group/view/ This is just an annoyance (one particular link leading people to the CSRF login page instead of directly to the action they requested) but the fix is easy and non-intrusive. Patch is: @@ -77,7 +77,8 @@ py:if test=can_sponsor dt${_('Add User:')}/dt dd - form action=${tg.url('/group/application_screen/%s' % group.name)} + form action=${tg.url('/group/application_screen/%s' % group.name)} +method=post input type='text' size='15' name='targetname'/ input type=submit value=${('Add')} / -Toshio signature.asc Description: OpenPGP digital signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change request -- fas template csrf fix
On 2009-03-12 08:05:45 AM, Toshio Kuratomi wrote: Patch is: @@ -77,7 +77,8 @@ py:if test=can_sponsor dt${_('Add User:')}/dt dd - form action=${tg.url('/group/application_screen/%s' % group.name)} + form action=${tg.url('/group/application_screen/%s' % group.name)} +method=post input type='text' size='15' name='targetname'/ input type=submit value=${('Add')} / +1 Thanks, Ricky pgpMuWLYTzsGG.pgp Description: PGP signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change request -- fas template csrf fix
On Thu, 12 Mar 2009, Toshio Kuratomi wrote: Found a template in fas that is not adding the csrf token properly. The Add User button on: https://admin.fedoraproject.org/accounts/group/view/ This is just an annoyance (one particular link leading people to the CSRF login page instead of directly to the action they requested) but the fix is easy and non-intrusive. Patch is: @@ -77,7 +77,8 @@ py:if test=can_sponsor dt${_('Add User:')}/dt dd - form action=${tg.url('/group/application_screen/%s' % group.name)} + form action=${tg.url('/group/application_screen/%s' % group.name)} +method=post input type='text' size='15' name='targetname'/ input type=submit value=${('Add')} / +1 -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change request - Using git send-email
Jeremy Katz wrote: Note that if it's a single patch, --compose is probably overkill as it makes two messages rather than one. Your commit message is self-explanatory, no? (... and if not, edit the 0001 file before running git send-email) For bonus points, the --subject-prefix option to git format-patch can be used to change the prefix from [PATCH] to [Change Request]: git format-patch --subject-prefix='Change Request' ... | git send-email (An alias in git could make this quite convenient.) Commentary could be added between the --- and the diffstat, as is common on the git and kernel lists for patches. -- ToddOpenPGP - KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp ~~ The best cure for insomnia is to get a lot of sleep. -- W.C. Fields pgpkKaITOOHz4.pgp Description: PGP signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Change Request -- python-fedora django auth fix
A problem was discovered with django auth and redirects. I'm spinning a new python-fedora package that has the fix for this and would like to deploy it on the hosts that we have transifex installed: app1 and app2 This package does not need to be installed in other places but it shouldn't hurt as the package just makes changes to the django auth provider which only transifex uses. Permission to install this? -Toshio signature.asc Description: OpenPGP digital signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change Request -- python-fedora django auth fix
On 2009-03-12 09:41:18 AM, Toshio Kuratomi wrote: A problem was discovered with django auth and redirects. I'm spinning a new python-fedora package that has the fix for this and would like to deploy it on the hosts that we have transifex installed: app1 and app2 This package does not need to be installed in other places but it shouldn't hurt as the package just makes changes to the django auth provider which only transifex uses. Permission to install this? +1 Thanks, Ricky pgpSzTgsrbQpK.pgp Description: PGP signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Meeting Today
Just a reminder that the our meetings are at 20:00 UTC. (you can run date -u to see what utc time it is). But this means for most of us in the states, the meetings are now an hour later, starting at 4:00 pm Eastern (3:00 pm Chicago Cubs time) -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Change Request
Can I get 2 +1's? ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
[Change Request] Added compose-x86/x86-8 to the ssh known hosts list
From: Mike McGrath mmcgr...@puppet1.fedora.phx.redhat.com This is technically a global change. Very low risk --- modules/ssh/files/ssh_known_hosts |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/ssh/files/ssh_known_hosts b/modules/ssh/files/ssh_known_hosts index 348b733..8f5543f 100644 --- a/modules/ssh/files/ssh_known_hosts +++ b/modules/ssh/files/ssh_known_hosts @@ -70,7 +70,7 @@ x86-4,10.8.34.223 ssh-rsa B3NzaC1yc2EBIwAAAQEAtXfXpXwocyfpDGPI1IbjhwuGHc x86-5,10.8.34.224 ssh-rsa B3NzaC1yc2EBIwAAAQEA9Ys1Mo5cBk8JIuMNcWHjIOqXvws0aHgVof1wj9XHcm1X0I37zmIlMAtGAzjuav3R9SqtvkNywqRTOW9OW4YcFUc3XmuWwkEfU1z2kZ9446Iof1YWiLS0i1qKQMHMcezZSwue16s6yTH+fk7ZDFNQXb/9kGT7rtDAG2tRhUwv/X39vhIhSyatB7j+uU/KWK90ymkBDDilALaANcPxJbfn3qsjBd+m4VpeBVdVj/QFRll0RCIas/Sc9irQpVXDDYYFl1zjhc+/NH/hR8GFTi+ojzLMmP18JKib0qtUF8YWjz1NzvVAf01BnfmMWhsQsZejmifsy6fjgHh1a0VuDqWFsQ== x86-6,10.8.34.225 ssh-rsa B3NzaC1yc2EBIwAAAQEAsSjsWRnCaE0Qg1XJSFVd0Yz1ZTl1u8fLmaSxXpJgrC1grH/EH/VzTFjBek/jA5UBUOcIwf8tQe7eBttcj0jSmaUyRU0k7bnPh6Ek+CJBbB0BZ5N50urcgLkjZJNdlRlYsfc8CJI9sl47IVSwZP3bPzNYDaVtHqcQKik04AluCew8gduvLiDjC/gVqVxSRwcgwHiMlkKWf7zlQS/yuzSBjzY0iMLrKaf0u89QO3trmfecBR7r2CuhKvxxam9DXSiuYxcLUW6hEDNvT2fpW4jFXZl9idpNjsJZXrLtNybAWvoEC/Sn2IBw0c6NfaqmNaxvKKjbfND8tYLf4AtdwoatZw== x86-7,10.8.34.226 ssh-rsa B3NzaC1yc2EBIwAAAQEAz1wkRNc9Tb/AxGBM0sLvcCxxpn9QNxpzdXEUZ/nQyGurBSMz8ZTBnGp2JlPwwmaJLkypR90nMQIUEgb4huF2JKQSSsv1l5kTxGiFwqo45zyu6gtwF78jl/AWvI91uAWrLAGspkVzaBRMhh5FQp3sCZGfYn1C+RYuoycH3MiPdHVPg3V0+32UTG444pwrhKJGY0yz4eproX/V6kxspCZ/sXvFZuSncm/rtYuxziktX7O8oYvmOgD/WhOk+ssxFM49L77hlgS/dbWkiYq1CeJklBlWYsZdw//nCu2VB3NZvY2aA3j7l0sqyI4xRQWlHhIbpBYTCRi2LvsXaE4O46ws1Q== -x86-8 ssh-rsa B3NzaC1yc2EBIwAAAQEAriDSchnp28etCJeeaBNTV3Szqi55lNPGNQND0HfGWG6KJAI8jhkabsMxgPNe9QO6UfwbMXmuIdqMfcqJGvjYf+hAZK2XWKe0AFuT2GFnIbRbWh1m9h1pe9NjNC40RGQ6aZXuutKTEhXExvmmWKf5neSDQlaqkcqRiKSRIz65nAcRrzVoG6J11OI00MPP8aHzlg1lGAX5Aj2liMt1ho0alzqX6V/Ndu2sdRmKo2BQZNZQr+GRYlat+Fcpj3sqpM4+wnulW5D7wTTAXYqFjcEA+YlPSlt9e3QBmiCohlXVydGMf6711YrEGUvtJctX64ZvcVkQH5oKp1QOcvMKwhX7UQ== +compose-x86,x86-8,10.8.34.227 ssh-rsa B3NzaC1yc2EBIwAAAQEAznKlEw9qRYsZ22VS2VO3tTxIk+WtuX4YooaQkXOoapX6IBZatRMFpSyeql/BeAniADLbWb3nKiZRCoAmKSMoIXKPfoF1dSRZMRwt4iR++9TYabuNbWEbBlV8aBfPtGuzAGhEBVHUsP3bHYwLPEFMJaomTB2biUAgFfk1RQn7JqsqsIMT7QfTIIMQD0HH+fqx7eeMTUFbS1l0m6h7H67mpZQNM++BZWXJQdY9+6VGdfOe2NAcasI1iABM+jXUE4f877QLCJEQjfGM2czUNm83DdpTkHKBUIMA8qtlKrixXYM7/pXB35i4R4OeZf9uj5cYjO5dM6Tm8bJsUee8XEFPAw== xen10,10.8.34.126 ssh-rsa B3NzaC1yc2EBIwAAAQEA5Oh5uaAbqBWxDyBbf81hMxO9fi7iGuiUhaO1lX9MqZ3I94eE7cJrG/lQ0cEUoXHaiX48iGT9avkBPyDf/gSEYHB6Gy1vrTi0u2aQMiM9RsCFy56m8+5Qn2H9GcWBhIXX5aqxfuucddoG6culHkPp69q+fpMYLlTi8jiVG6X0ElR8bC5Msr8g1cidkLn6vhMWImGpE+6465/LhVvh8B8BhylQIQRjN4DxmITcZjvLFCAApM9FqhxOJWDX+e9maBvBd9sLCYRFZeJqBy34OZu37W2pKTWzpDTEZSyZcHKggZLA/9IiVJcBPvNwnPMExw56nJBoJc9xwueAaaIEbJB1Mw== xen11,10.8.34.209 ssh-rsa B3NzaC1yc2EBIwAAAQEA3NGlyoMcpLss5zVnvTImNGvCtfKe38vJKtr5swRoYC9B674rCzpUfgpLEIMq99AXrMUu9oH6LhrO9TGMewab6rsN0BQbpCMr4pYdZbuQSC2DRBEjpgaCDNhqy5usyaPIy1vzt/Vwi8I0gBYFd8sfA6JJ7++k1v/RoCeil5+BRgBRKiq1dLjL3LhdL5vUGU/60VrhQIkYm+hILZAqk9cAELIg622L8enbGBoKIxB9pmpAqnn4Kp0Nm35Y3wUSiqVcloup7ITg8rLwEGXUsy9CRz2b9JFNJAJNBTGp613Tf11Bc4f9LRF0yItgNMWnFXRw7ziEsKzRKQ0V9VG7hEeLhw== xen12,10.8.34.201 ssh-rsa B3NzaC1yc2EBIwAAAQEA3h0fK6aExXOGPlIv3SlTjBtDVh8XaT759k+7xcqYI/MUm7W/7PSPJE3V/C51+7chqhlSiwHcpzB36EnePnakip/VrjxSEgaFD7NIb43AA5us8qC/fO/UUq1XpE2Qcl1uWP7EZlcsjAnrKQ5DaIy8FgvkpugVCFgUV+NSJVxI94s73IYaLa0cqKg/OsMBU8mVRncF9MdbbJgCW/5+JCogON0CyYU8+8RKmPrMcEs56qbe03M0LP0NZ0cjSHnxbeDI0+aGnrHdPDvQHRA7/TkkvluZ8Ec3A9duaLq6F2GSePEuqhvnMdjLoXsPuzW6vUVJLyfW4rLssh05QHgRl0fhUQ== -- 1.5.5.6 ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change Request
On Thu, 12 Mar 2009, Mike McGrath wrote: Can I get 2 +1's? U, clearly I missed a step here :) -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
[Change Request] Adding files to ssh-known-hsts
Trying again, hopefully with less fail. -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
[PATCH] Added compose-x86/x86-8 to the ssh known hosts list
From: Mike McGrath mmcgr...@puppet1.fedora.phx.redhat.com This is technically a global change. Very low risk --- modules/ssh/files/ssh_known_hosts |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/ssh/files/ssh_known_hosts b/modules/ssh/files/ssh_known_hosts index 348b733..8f5543f 100644 --- a/modules/ssh/files/ssh_known_hosts +++ b/modules/ssh/files/ssh_known_hosts @@ -70,7 +70,7 @@ x86-4,10.8.34.223 ssh-rsa B3NzaC1yc2EBIwAAAQEAtXfXpXwocyfpDGPI1IbjhwuGHc x86-5,10.8.34.224 ssh-rsa B3NzaC1yc2EBIwAAAQEA9Ys1Mo5cBk8JIuMNcWHjIOqXvws0aHgVof1wj9XHcm1X0I37zmIlMAtGAzjuav3R9SqtvkNywqRTOW9OW4YcFUc3XmuWwkEfU1z2kZ9446Iof1YWiLS0i1qKQMHMcezZSwue16s6yTH+fk7ZDFNQXb/9kGT7rtDAG2tRhUwv/X39vhIhSyatB7j+uU/KWK90ymkBDDilALaANcPxJbfn3qsjBd+m4VpeBVdVj/QFRll0RCIas/Sc9irQpVXDDYYFl1zjhc+/NH/hR8GFTi+ojzLMmP18JKib0qtUF8YWjz1NzvVAf01BnfmMWhsQsZejmifsy6fjgHh1a0VuDqWFsQ== x86-6,10.8.34.225 ssh-rsa B3NzaC1yc2EBIwAAAQEAsSjsWRnCaE0Qg1XJSFVd0Yz1ZTl1u8fLmaSxXpJgrC1grH/EH/VzTFjBek/jA5UBUOcIwf8tQe7eBttcj0jSmaUyRU0k7bnPh6Ek+CJBbB0BZ5N50urcgLkjZJNdlRlYsfc8CJI9sl47IVSwZP3bPzNYDaVtHqcQKik04AluCew8gduvLiDjC/gVqVxSRwcgwHiMlkKWf7zlQS/yuzSBjzY0iMLrKaf0u89QO3trmfecBR7r2CuhKvxxam9DXSiuYxcLUW6hEDNvT2fpW4jFXZl9idpNjsJZXrLtNybAWvoEC/Sn2IBw0c6NfaqmNaxvKKjbfND8tYLf4AtdwoatZw== x86-7,10.8.34.226 ssh-rsa B3NzaC1yc2EBIwAAAQEAz1wkRNc9Tb/AxGBM0sLvcCxxpn9QNxpzdXEUZ/nQyGurBSMz8ZTBnGp2JlPwwmaJLkypR90nMQIUEgb4huF2JKQSSsv1l5kTxGiFwqo45zyu6gtwF78jl/AWvI91uAWrLAGspkVzaBRMhh5FQp3sCZGfYn1C+RYuoycH3MiPdHVPg3V0+32UTG444pwrhKJGY0yz4eproX/V6kxspCZ/sXvFZuSncm/rtYuxziktX7O8oYvmOgD/WhOk+ssxFM49L77hlgS/dbWkiYq1CeJklBlWYsZdw//nCu2VB3NZvY2aA3j7l0sqyI4xRQWlHhIbpBYTCRi2LvsXaE4O46ws1Q== -x86-8 ssh-rsa B3NzaC1yc2EBIwAAAQEAriDSchnp28etCJeeaBNTV3Szqi55lNPGNQND0HfGWG6KJAI8jhkabsMxgPNe9QO6UfwbMXmuIdqMfcqJGvjYf+hAZK2XWKe0AFuT2GFnIbRbWh1m9h1pe9NjNC40RGQ6aZXuutKTEhXExvmmWKf5neSDQlaqkcqRiKSRIz65nAcRrzVoG6J11OI00MPP8aHzlg1lGAX5Aj2liMt1ho0alzqX6V/Ndu2sdRmKo2BQZNZQr+GRYlat+Fcpj3sqpM4+wnulW5D7wTTAXYqFjcEA+YlPSlt9e3QBmiCohlXVydGMf6711YrEGUvtJctX64ZvcVkQH5oKp1QOcvMKwhX7UQ== +compose-x86,x86-8,10.8.34.227 ssh-rsa B3NzaC1yc2EBIwAAAQEAznKlEw9qRYsZ22VS2VO3tTxIk+WtuX4YooaQkXOoapX6IBZatRMFpSyeql/BeAniADLbWb3nKiZRCoAmKSMoIXKPfoF1dSRZMRwt4iR++9TYabuNbWEbBlV8aBfPtGuzAGhEBVHUsP3bHYwLPEFMJaomTB2biUAgFfk1RQn7JqsqsIMT7QfTIIMQD0HH+fqx7eeMTUFbS1l0m6h7H67mpZQNM++BZWXJQdY9+6VGdfOe2NAcasI1iABM+jXUE4f877QLCJEQjfGM2czUNm83DdpTkHKBUIMA8qtlKrixXYM7/pXB35i4R4OeZf9uj5cYjO5dM6Tm8bJsUee8XEFPAw== xen10,10.8.34.126 ssh-rsa B3NzaC1yc2EBIwAAAQEA5Oh5uaAbqBWxDyBbf81hMxO9fi7iGuiUhaO1lX9MqZ3I94eE7cJrG/lQ0cEUoXHaiX48iGT9avkBPyDf/gSEYHB6Gy1vrTi0u2aQMiM9RsCFy56m8+5Qn2H9GcWBhIXX5aqxfuucddoG6culHkPp69q+fpMYLlTi8jiVG6X0ElR8bC5Msr8g1cidkLn6vhMWImGpE+6465/LhVvh8B8BhylQIQRjN4DxmITcZjvLFCAApM9FqhxOJWDX+e9maBvBd9sLCYRFZeJqBy34OZu37W2pKTWzpDTEZSyZcHKggZLA/9IiVJcBPvNwnPMExw56nJBoJc9xwueAaaIEbJB1Mw== xen11,10.8.34.209 ssh-rsa B3NzaC1yc2EBIwAAAQEA3NGlyoMcpLss5zVnvTImNGvCtfKe38vJKtr5swRoYC9B674rCzpUfgpLEIMq99AXrMUu9oH6LhrO9TGMewab6rsN0BQbpCMr4pYdZbuQSC2DRBEjpgaCDNhqy5usyaPIy1vzt/Vwi8I0gBYFd8sfA6JJ7++k1v/RoCeil5+BRgBRKiq1dLjL3LhdL5vUGU/60VrhQIkYm+hILZAqk9cAELIg622L8enbGBoKIxB9pmpAqnn4Kp0Nm35Y3wUSiqVcloup7ITg8rLwEGXUsy9CRz2b9JFNJAJNBTGp613Tf11Bc4f9LRF0yItgNMWnFXRw7ziEsKzRKQ0V9VG7hEeLhw== xen12,10.8.34.201 ssh-rsa B3NzaC1yc2EBIwAAAQEA3h0fK6aExXOGPlIv3SlTjBtDVh8XaT759k+7xcqYI/MUm7W/7PSPJE3V/C51+7chqhlSiwHcpzB36EnePnakip/VrjxSEgaFD7NIb43AA5us8qC/fO/UUq1XpE2Qcl1uWP7EZlcsjAnrKQ5DaIy8FgvkpugVCFgUV+NSJVxI94s73IYaLa0cqKg/OsMBU8mVRncF9MdbbJgCW/5+JCogON0CyYU8+8RKmPrMcEs56qbe03M0LP0NZ0cjSHnxbeDI0+aGnrHdPDvQHRA7/TkkvluZ8Ec3A9duaLq6F2GSePEuqhvnMdjLoXsPuzW6vUVJLyfW4rLssh05QHgRl0fhUQ== -- 1.5.5.6 ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: [PATCH] Added compose-x86/x86-8 to the ssh known hosts list
On Thu, 2009-03-12 at 18:41 +, Mike McGrath wrote: This is technically a global change. Very low risk Diffs of this file are nearly impossible to read, but I'll trust what you're doing rather than the diff. +1 -- Jesse Keating Fedora -- FreedomĀ² is a feature! identi.ca: http://identi.ca/jkeating signature.asc Description: This is a digitally signed message part ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Meeting Today
On Thu, 2009-03-12 at 13:19 -0500, Mike McGrath wrote: (3:00 pm Chicago Cubs time) Chicago Cubs time, is that when everybody drinks because the cubs lost again? -- Jesse Keating Fedora -- FreedomĀ² is a feature! identi.ca: http://identi.ca/jkeating signature.asc Description: This is a digitally signed message part ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Wordpress?
Bret McMillan wrote: Rahul Sundaram wrote: Clint Savage wrote: Mike, Do you have a link to the mailing list thread? I'd like to read up on it. I like MU, don't get me wrong, just wonder why it was chosen. We wanted it for a Fedora News site. Refer https://fedorahosted.org/fedora-infrastructure/ticket/178 At this point, I think we're blocked on a theme (outside my skillset). I think jonrob was going to look at this time-permitting. Jon Rob won't have time for this for a while. http://jonrob.wordpress.com/2009/03/09/time-out/ Mo, if you can get a theme going, I would like to launch this news site soon, ahead of Fedora 11. Thanks for the help. Rahul ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: [Change Request] Added compose-x86/x86-8 to the ssh known hosts list
On 2009-03-12 06:37:20 PM, Mike McGrath wrote: From: Mike McGrath mmcgr...@puppet1.fedora.phx.redhat.com This is technically a global change. Very low risk --- modules/ssh/files/ssh_known_hosts |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) +1 Thanks, Ricky pgp613oCXC0e0.pgp Description: PGP signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change Request
On Thu, Mar 12, 2009 at 12:37 PM, Mike McGrath mmcgr...@redhat.com wrote: On Thu, 12 Mar 2009, Mike McGrath wrote: Can I get 2 +1's? U, clearly I missed a step here :) +1 -- Stephen J Smoogen. -- BSD/GNU/Linux How far that little candle throws his beams! So shines a good deed in a naughty world. = Shakespeare. The Merchant of Venice ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Meeting Today
2009/3/12 Jesse Keating jkeat...@redhat.com: On Thu, 2009-03-12 at 13:19 -0500, Mike McGrath wrote: (3:00 pm Chicago Cubs time) Chicago Cubs time, is that when everybody drinks because the cubs lost again? They drink whether or not the Cubs won. The issue is it is Harry Caray time or not. -- Stephen J Smoogen. -- BSD/GNU/Linux How far that little candle throws his beams! So shines a good deed in a naughty world. = Shakespeare. The Merchant of Venice ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: [Change Request] Adding files to ssh-known-hsts
On Thu, Mar 12, 2009 at 12:41 PM, Mike McGrath mmcgr...@redhat.com wrote: Trying again, hopefully with less fail. What hosts? -- Stephen J Smoogen. -- BSD/GNU/Linux How far that little candle throws his beams! So shines a good deed in a naughty world. = Shakespeare. The Merchant of Venice ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Meeting Log - 2009-03-12
20:00 -!- mmcgrath changed the topic of #fedora-meeting to: Infrastructure -- Who's here? 20:00 * ricky 20:00 * collier_s is here 20:01 * ivazquez|laptop is around 20:01 -!- MostafaDaneshvar [n=mosta...@unaffiliated/mostafadaneshvar] has joined #fedora-meeting 20:01 * SmootherFrOgZ here 20:02 mmcgrath Ok, so lets get started 20:03 -!- Sonar_Gal [n=and...@fedora/SonarGal] has quit Connection timed out 20:03 -!- mmcgrath changed the topic of #fedora-meeting to: Infrastructure -- Tickets 20:03 * skvidal is 20:03 mmcgrath Looks like there actually aren't any tickets. 20:04 mmcgrath So next topic 20:04 -!- mmcgrath changed the topic of #fedora-meeting to: Infrastructure -- Fedora Cloud 20:04 mmcgrath So it looks like our Fedora cloud might finally be delivered. 20:04 SmootherFrOgZ \o/ 20:04 mmcgrath From what I understand the last of the network stuff should be ready by tomorrow night. 20:05 mmcgrath SmootherFrOgZ: you still interested.. even though it's been 2 months?!?! :) 20:05 SmootherFrOgZ mmcgrath: hahaha, are you kidding me ? 20:06 ricky What work is there to be done once it's delivered? 20:06 mmcgrath :) 20:06 SmootherFrOgZ i've a lot of work to commit ;) 20:06 mmcgrath ricky: my understanding is we've been waiting for months to get a switch and router configured. 20:06 ricky Haha 20:06 mmcgrath anywho. 20:06 mmcgrath It's coming! 20:06 mmcgrath so that's good. 20:06 mmcgrath And it's something we can work on while we're frozen so that's nice too. 20:06 -!- basilgohar [n=basil...@60.48.61.220] has quit Read error: 54 (Connection reset by peer) 20:06 -!- mmcgrath changed the topic of #fedora-meeting to: Infrastructure -- Password Resets. 20:06 collier_s i'd like to help out in anyway / shape / form 20:07 mmcgrath collier_s: sure thing, I'm sure there will be a bunch of stuff to do. 20:07 mmcgrath So the password resets went remarkably well for most people. There's a pretty vocal minority though. 20:07 mmcgrath Some good ideas came out of it. 20:07 -!- tibbs [n=ti...@fedora/tibbs] has quit Konversation terminated! 20:08 ricky So exactly what changes are we looking at for next time? 20:08 mmcgrath ricky: the 3 things mentioned in my email. 20:08 mmcgrath recovery of home dir, more explicit email, and aliases to stick around longer. 20:08 ricky Sorry, I've been looking at several different threads on this topic 20:09 ricky Aha 20:09 mmcgrath And since people bitched and moaned enough we will probably go off of last seen and start updating that. 20:09 mmcgrath It accomplishes a similar goal. It'll make things easier on the package contributors if we can get it updated in pkgdb and bodhi. 20:10 * ricky will look at getting fasClient and the expiry script modified for those things 20:10 mmcgrath and since they seem to be the ones who had the biggest problem figuring out who this whole process worked, if we can avoid inconveniencing them in the future we might as well. 20:10 mmcgrath s/who/how/ 20:10 -!- basilgohar [n=basil...@220.61.48.60.trm01-home.tm.net.my] has joined #fedora-meeting 20:11 mmcgrath ricky: how much time will you have to devote to that over the next month or so? 20:11 mmcgrath I'd like to enable a regular daily check as soon as we can. 20:11 mmcgrath abadger1999 is familiar with some of the issues and suggestions as well but I didn't want to volunteer him to fix them. 20:11 ricky I wish I could give a definitely number :-( I should be able to get the big requirements done this week, while it's spring break 20:11 ricky **definite 20:12 mmcgrath ricky: k, well keep me in the loop. If things fall apart I can spend some more time getting it up and going. 20:12 mmcgrath Anyone have anything else on that? 20:13 -!- mmcgrath changed the topic of #fedora-meeting to: Infrastructure -- compose-x86 20:13 mmcgrath Boy was that box borked. 20:13 mmcgrath it's mostly fixed now. 20:13 mmcgrath as a result we're also going to do bios updates to our other x86 blades and the ppc blades just for fun. 20:13 mmcgrath compose-x86 runs rawhide so we had all sorts of fun getting it all back up. 20:13 -!- mmcgrath changed the topic of #fedora-meeting to: Infrastructure -- Transifex 20:14 * mmcgrath summons glezos 20:14 mmcgrath ivazquez|laptop: how's transifex going? 20:14 ivazquez|laptop Well. 20:14 mmcgrath My understanding is we couldn't commit last night because the transif user didn't have access. 20:14 mmcgrath other then that and the final migration to a $REAL_DB, things are working? 20:15 ivazquez|laptop A fix for PG 8.2+ has been applied. 20:15 ivazquez|laptop MySQL we all know about. 20:15 mmcgrath ivazquez|laptop: that was a transifex fix, not a django fix right? 20:15 ivazquez|laptop Correct. 20:15 -!- Zool^ [n=kal...@19.81-166-29.customer.lyse.net] has quit Read error: 110 (Connection timed out) 20:15 mmcgrath k 20:15 mmcgrath do you know when this is going to start getting used to do actual translations?
Unclaimed F11 Beta Tickets
Hey all, just a heads up. Some of these tickets are still unclaimed (I just created them today). Some of these (like the MM redirects) can only be done by people who have access to them. But others, like the website, could be done by anyone who can email a patch our way. Though ricky usually does it :) https://fedorahosted.org/fedora-infrastructure/report/9 -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Change request -- Django Provider update 2
Okay, the last auth update fixed redirects but broke logging into django apps. I'd like to install a new python-fedora with a one-line change for that. === modified file 'fedora/django/auth/middleware.py' --- fedora/django/auth/middleware.py2009-03-12 14:02:58 + +++ fedora/django/auth/middleware.py2009-03-12 22:34:37 + @@ -39,7 +39,7 @@ logout(request) def process_response(self, request, response): -if type(response) == HttpResponse: +if response.status_code != 301: if isinstance(request.user, AnonymousUser): #response.set_cookie(key='tg-visit', value='', max_age=0) if 'tg-visit' in request.session: -Toshio signature.asc Description: OpenPGP digital signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change request -- Django Provider update 2
On Thu, 12 Mar 2009, Toshio Kuratomi wrote: Okay, the last auth update fixed redirects but broke logging into django apps. I'd like to install a new python-fedora with a one-line change for that. === modified file 'fedora/django/auth/middleware.py' --- fedora/django/auth/middleware.py 2009-03-12 14:02:58 + +++ fedora/django/auth/middleware.py 2009-03-12 22:34:37 + @@ -39,7 +39,7 @@ logout(request) def process_response(self, request, response): -if type(response) == HttpResponse: +if response.status_code != 301: if isinstance(request.user, AnonymousUser): #response.set_cookie(key='tg-visit', value='', max_age=0) if 'tg-visit' in request.session: +1 -Mike ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: Change request -- Django Provider update 2
On 2009-03-12 05:06:45 PM, Toshio Kuratomi wrote: Okay, the last auth update fixed redirects but broke logging into django apps. I'd like to install a new python-fedora with a one-line change for that. === modified file 'fedora/django/auth/middleware.py' --- fedora/django/auth/middleware.py 2009-03-12 14:02:58 + +++ fedora/django/auth/middleware.py 2009-03-12 22:34:37 + @@ -39,7 +39,7 @@ logout(request) def process_response(self, request, response): -if type(response) == HttpResponse: +if response.status_code != 301: if isinstance(request.user, AnonymousUser): #response.set_cookie(key='tg-visit', value='', max_age=0) if 'tg-visit' in request.session: +1 Thanks, Ricky pgptJa6eRE325.pgp Description: PGP signature ___ Fedora-infrastructure-list mailing list Fedora-infrastructure-list@redhat.com https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list
Re: tiket #72
Hello. It is channges for the dispatcher.py. I added the function remove_user(). === modified file 'pkgdb/dispatcher.py' --- pkgdb/dispatcher.py 2009-02-27 15:58:55 + +++ pkgdb/dispatcher.py 2009-03-13 01:20:10 + @@ -1397,3 +1397,95 @@ identity.current.user, [clone_branch]) return dict(pkglisting=clone_branch) + +@expose(allow_json=True) +# Check that the requestor is in a group that could potentially set ACLs. +@identity.require(identity.not_anonymous()) +def remove_user(self, pkg_name, username, collectn_list=None): + '''Remove users from a package. +:arg pkg_name: Name of the package +:arg username: Name of user to remove from the package +:arg collectn_list: list of collections like 'F-10', 'devel' +''' +person = fas.person_by_username(username) +if not person: +return dict(status=False, +message='Specified user name %s does not have a' \ +' Fedora Account' % username) +try: +# pylint: disable-msg=E1101 +pkg = Package.query.filter_by(name=pkg_name).one() + except InvalidRequestError: + return dict(status=False, message='Package %s does not exist' % pkg_name) + +# Check that the current user is allowed to change acl statuses +approved = self._user_can_set_acls(identity, pkg) +if not ident.in_group('cvsadmin'): +return dict(status=False, message= +'%s is not allowed to remove user from the package' % +identity.current.user_name) + +log_msgs = [] + + if collectn_list: + for simple_name in collectn_list: + try: + collectn = Collection.by_simple_name(simple_name) + except InvalidRequestError: + return dict(status=False, message='Collection %s does not exist' % simple_name) + +pkg_listing = PackageListing.query.filter_by(packageid=pkg.id, + collectionid=collectn.id).one() + +acls = PersonPackageListingAcl.query.filter(and_( + PersonPackageListingAcl.c.personpackagelistingid + == PersonPackageListing.c.id, + PersonPackageListing.c.packagelistingid == pkg_listing.id, + PersonPackageListing.c.username == person['username'])).all() + +for acl in acls: + person_acl = self._create_or_modify_acl(pkg_listing, person['id'], acl, self.obsoleteStatus) + + log_msg = u'%s has set the %s acl on %s (%s %s) to Obsolete for %s' % ( +identity.current.user_name, acl, pkg.name, +pkg_listing.collection.name, pkg_listing.collection.version, +person['username']) + log = PersonPackageListingAclLog(identity.current.user.id, +self.obsoleteStatus.statuscodeid, log_msg) + log.acl = person_acl # pylint: disable-msg=W0201 + log_msgs.append(log_msg) + + else: +for pkg_listing in pkg.listings: + acls = PersonPackageListingAcl.query.filter(and_( + PersonPackageListingAcl.c.personpackagelistingid + == PersonPackageListing.c.id, + PersonPackageListing.c.packagelistingid == pkg_listing.id, + PersonPackageListing.c.username == person['username'])).all() + +for acl in acls: +person_acl = self._create_or_modify_acl(pkg_listing, person['id'], acl, self.obsoleteStatus) + +log_msg = u'%s has set the %s acl on %s (%s %s) to Obsolete for %s' % ( +identity.current.user_name, acl, pkg.name, +pkg_listing.collection.name, pkg_listing.collection.version, +person['username']) +log = PersonPackageListingAclLog(identity.current.user.id, +self.obsoleteStatus.statuscodeid, log_msg) +log.acl = person_acl # pylint: disable-msg=W0201 +log_msgs.append(log_msg) + +try: +session.flush() +except SQLError, e: +# An error was generated +return dict(status=False, +message='Not able to change acl %s on %s with status %s' \ +% (acl, pkgid,
Re: tiket #72
Very nice work! I have a few comments; mostly based on new coding styles that we're enforcing in new code but haven't made it into old code yet. Dmitry Kolesov wrote: Hello. It is channges for the dispatcher.py. I added the function remove_user(). === modified file 'pkgdb/dispatcher.py' --- pkgdb/dispatcher.py 2009-02-27 15:58:55 + +++ pkgdb/dispatcher.py 2009-03-13 01:20:10 + @@ -1397,3 +1397,95 @@ identity.current.user, [clone_branch]) return dict(pkglisting=clone_branch) + +@expose(allow_json=True) +# Check that the requestor is in a group that could potentially set ACLs. +@identity.require(identity.not_anonymous()) +def remove_user(self, pkg_name, username, collectn_list=None): I'd reorder the arguments to be username, pkg_name, collectn_list. This is because username is the subject of the action so it's more important than the pkg_name and collectn_list. (Also, the pkg_name and collectn_list work together to define the pkglistings that the user is being removed from). + '''Remove users from a package. +:arg pkg_name: Name of the package +:arg username: Name of user to remove from the package +:arg collectn_list: list of collections like 'F-10', 'devel' +''' I'd add the default value to the collectn_list documentation and what it means. + person = fas.person_by_username(username) +if not person: +return dict(status=False, +message='Specified user name %s does not have a' \ +' Fedora Account' % username) Since this is a removal, we don't need to retrieve the person information from fas. The username should be sufficient. +try: +# pylint: disable-msg=E1101 +pkg = Package.query.filter_by(name=pkg_name).one() + except InvalidRequestError: + return dict(status=False, message='Package %s does not exist' % pkg_name) We're trying to move to a new style of returning errors. It's documented here: https://fedorahosted.org/releases/p/y/python-fedora/doc/service.html#error-handling Basically, when returning an error you'll do something like this: from turbogears import flash [...] flash('Package %s does not exist' % pkg_name) return dict(exc='NoPackageError') When the client receives this it will see that exc is set and create an AppError exception with the name in exc and the message that you called flash() with (it will end up in tg_flash). +# Check that the current user is allowed to change acl statuses +approved = self._user_can_set_acls(identity, pkg) +if not ident.in_group('cvsadmin'): I think we've abstracted 'cvsadmin' out to a config file option. You should be able to do this: from pkgdb.utils import admin_grp [...] if not identity.in_group(admin_grp): Also, I think you want identity rather than ident +return dict(status=False, message= +'%s is not allowed to remove user from the package' % +identity.current.user_name) + Same thing about returning errors here. +log_msgs = [] + + if collectn_list: + for simple_name in collectn_list: + try: + collectn = Collection.by_simple_name(simple_name) + except InvalidRequestError: + return dict(status=False, message='Collection %s does not exist' % simple_name) + Same thing about returning errors +pkg_listing = PackageListing.query.filter_by(packageid=pkg.id, + collectionid=collectn.id).one() + +acls = PersonPackageListingAcl.query.filter(and_( + PersonPackageListingAcl.c.personpackagelistingid + == PersonPackageListing.c.id, + PersonPackageListing.c.packagelistingid == pkg_listing.id + PersonPackageListing.c.username == person['username'])).all() + You can change this from person['username'] to username +for acl in acls: + person_acl = self._create_or_modify_acl(pkg_listing, person['id'], acl, self.obsoleteStatus) + maploin has just committed a new db schema and code to the db that changes things like this to use username instead of id. So you can just pass username instead of person['id']. + log_msg = u'%s has set the %s acl on %s (%s %s) to Obsolete for %s' % ( +identity.current.user_name, acl, pkg.name, +pkg_listing.collection.name, pkg_listing.collection.version, +person['username']) + log =