Hi all,
Justin has asked me if we could increase the maximal size limit for upload of kernel tests results (from 10 Kb to 25 Kb). He has done it upstream, this commit changes it in prod.
I would like to apply it and run the corresponding playbook.
+1s?
Thanks, Pierre
Changed upstream at https://github.com/jmflinuxtx/kerneltest-harness/commit/286278c6b0269f39ff1f... --- roles/kerneltest/templates/kerneltest.cfg | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/roles/kerneltest/templates/kerneltest.cfg b/roles/kerneltest/templates/kerneltest.cfg index 6552cb1..e15785e 100644 --- a/roles/kerneltest/templates/kerneltest.cfg +++ b/roles/kerneltest/templates/kerneltest.cfg @@ -26,7 +26,7 @@ ADMIN_GROUP = ['sysadmin-kernel', 'sysadmin-main'] ALLOWED_MIMETYPES = ['text/plain', 'text/x-log']
# Restrict the size of content uploaded, this is 10Kb -MAX_CONTENT_LENGTH = 1024 * 10 +MAX_CONTENT_LENGTH = 1024 * 25
# This is required to fix login
On Wed, Feb 25, 2015 at 03:44:27PM +0100, Pierre-Yves Chibon wrote:
diff --git a/roles/kerneltest/templates/kerneltest.cfg b/roles/kerneltest/templates/kerneltest.cfg index 6552cb1..e15785e 100644 --- a/roles/kerneltest/templates/kerneltest.cfg +++ b/roles/kerneltest/templates/kerneltest.cfg @@ -26,7 +26,7 @@ ADMIN_GROUP = ['sysadmin-kernel', 'sysadmin-main'] ALLOWED_MIMETYPES = ['text/plain', 'text/x-log']
# Restrict the size of content uploaded, this is 10Kb -MAX_CONTENT_LENGTH = 1024 * 10 +MAX_CONTENT_LENGTH = 1024 * 25
You should probably also change the comment to match...
On Wed, Feb 25, 2015 at 09:46:32AM -0500, Chuck Anderson wrote:
On Wed, Feb 25, 2015 at 03:44:27PM +0100, Pierre-Yves Chibon wrote:
diff --git a/roles/kerneltest/templates/kerneltest.cfg b/roles/kerneltest/templates/kerneltest.cfg index 6552cb1..e15785e 100644 --- a/roles/kerneltest/templates/kerneltest.cfg +++ b/roles/kerneltest/templates/kerneltest.cfg @@ -26,7 +26,7 @@ ADMIN_GROUP = ['sysadmin-kernel', 'sysadmin-main'] ALLOWED_MIMETYPES = ['text/plain', 'text/x-log']
# Restrict the size of content uploaded, this is 10Kb -MAX_CONTENT_LENGTH = 1024 * 10 +MAX_CONTENT_LENGTH = 1024 * 25
You should probably also change the comment to match...
Good point, I'll do that in a second commit (that I'll push at the same time if I get 2 +1).
Thanks, Pierre
On Wed, 25 Feb 2015 15:49:45 +0100 Pierre-Yves Chibon pingou@pingoured.fr wrote:
On Wed, Feb 25, 2015 at 09:46:32AM -0500, Chuck Anderson wrote:
On Wed, Feb 25, 2015 at 03:44:27PM +0100, Pierre-Yves Chibon wrote:
diff --git a/roles/kerneltest/templates/kerneltest.cfg b/roles/kerneltest/templates/kerneltest.cfg index 6552cb1..e15785e 100644 --- a/roles/kerneltest/templates/kerneltest.cfg +++ b/roles/kerneltest/templates/kerneltest.cfg @@ -26,7 +26,7 @@ ADMIN_GROUP = ['sysadmin-kernel', 'sysadmin-main'] ALLOWED_MIMETYPES = ['text/plain', 'text/x-log'] # Restrict the size of content uploaded, this is 10Kb -MAX_CONTENT_LENGTH = 1024 * 10 +MAX_CONTENT_LENGTH = 1024 * 25
You should probably also change the comment to match...
Good point, I'll do that in a second commit (that I'll push at the same time if I get 2 +1).
I assume this was also requested by the kernel folks, as well.
+1 on setting and comment change
Tim
infrastructure@lists.fedoraproject.org