add/setLimitRequestBody parameter to 10G

[mcgonago]

Add/Set LimitRequestBody parameter to 10G

This is to get around the 1G limit [1]

[1] https://access.redhat.com/articles/6975397

Jira: https://issues.redhat.com/browse/OSPRH-17359

[stuggi]

@mcgonago maybe I should rebase #402 that you can customize the vhost? we have the same landed in keystone to configure OIDC Federation settings. openstack-k8s-operators/keystone-operator#525

[mcgonago]

I tested this patch and see LimitRequestBody 10737418240 showing up in the right place in /etc/httpd/conf/httpd.conf

ServerTokens Prod
ServerSignature Off
TraceEnable Off
.
.
.
<VirtualHost *:8443>
  ## Vhost docroot
  DocumentRoot "/var/www/"
  .
  .
  .
  ## WSGI configuration
  WSGIApplicationGroup %{GLOBAL}
  WSGIDaemonProcess apache display-name=horizon group=apache processes=4 threads=1 user=apache
  WSGIProcessGroup apache
  WSGIScriptAlias /dashboard "/usr/share/openstack-dashboard/openstack_dashboard/wsgi.py"

  ## Extend LimitReqeustBody to 10GB
  LimitRequestBody 10737418240
</VirtualHost>

[stuggi]

If the idea is to set the LimitRequestBody to a fixed const, why not just add it to https://github.com/openstack-k8s-operators/horizon-operator/blob/main/templates/horizon/config/httpd.conf ?

I have rebased #402, please have a look, which would be adding the same approach to horizon to be able to customize the httpd vhost.

[mcgonago]

@stuggi - good point. I updated the changes to only have the setting in httpd.conf

[stuggi]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mcgonago, stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [mcgonago,stuggi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[stuggi]

/cherry-pick 18.0-fr3

[openshift-cherrypick-robot]

@stuggi: once the present PR merges, I will cherry-pick it on top of 18.0-fr3 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick 18.0-fr3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[openshift-cherrypick-robot]

@stuggi: new pull request created: #486

Details

In response to this:

/cherry-pick 18.0-fr3

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.