Uploaded image for project: 'Couchbase Kubernetes'
  1. Couchbase Kubernetes
  2. K8S-2504

Link out to OCP MachineConfig docs

    XMLWordPrintable

Details

    • Page
    • Status: Resolved
    • Minor
    • Resolution: Resolved
    • None
    • 2.3.0, 2.2.1
    • documentation
    • None
    • 1

    Description

      When discussing best practices and guidelines here: https://docs.couchbase.com/operator/current/best-practices.html#ulimits

      We should link out to the OCP documentation to resolve the 1024 pid limit by default: https://access.redhat.com/solutions/5366631 

      https://docs.openshift.com/container-platform/4.9/nodes/nodes/nodes-nodes-managing.html 

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          Ascending order - Click to sort in descending order
          patrick.stephens Patrick Stephens (Inactive) created issue -
          patrick.stephens Patrick Stephens (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to CBSE-10914 [ CBSE-10914 ]
          patrick.stephens Patrick Stephens (Inactive) made changes -
          Link This issue relates to CBSE-10337 [ CBSE-10337 ]
          dfinlay Dave Finlay added a comment -

          Note that that OCP docs link is considered "subscriber exclusive content", so general walking around folks won't be able to read it.

          dfinlay Dave Finlay added a comment - Note that that OCP docs link is considered "subscriber exclusive content", so general walking around folks won't be able to read it.
          simon.murray Simon Murray made changes -
          Rank Ranked higher
          simon.murray Simon Murray made changes -
          Rank Ranked higher
          simon.murray Simon Murray made changes -
          Rank Ranked higher
          simon.murray Simon Murray made changes -
          Rank Ranked higher
          simon.murray Simon Murray made changes -
          Rank Ranked higher
          ingenthr Matt Ingenthron made changes -
          Assignee Roshani Sanghavi [ JIRAUSER24738 ] Anil Kumar [ anil ]
          anil Anil Kumar (Inactive) added a comment -

          https://github.com/couchbase/couchbase-operator/pull/29

          anil Anil Kumar (Inactive) added a comment - https://github.com/couchbase/couchbase-operator/pull/29
          anil Anil Kumar (Inactive) added a comment -

          Maria Shodunke could you please help push these changes to https://docs.couchbase.com/operator/2.3-beta/best-practices.html#ulimits

           

          Here are the changes - 

          1. right below the first paragraph, add this caution
            [CAUTION]
              ====
              Red Hat OpenShift 4.0 and above sets the default limit of the maximum number of process IDs in the cgroup to be 1024; you can read about this here https://access.redhat.com/solutions/5366631 (Red Hat subscription required). A workaround is to increase the pids_limit in both the KubeletConfig and ContainerRuntimeConfig.
              ====

           

          see this commit for reference - https://github.com/couchbase/couchbase-operator/pull/29/commits/689a40ed9273a5172528e9b7ea169aae9ccd8ff8 

          anil Anil Kumar (Inactive) added a comment - Maria Shodunke could you please help push these changes to https://docs.couchbase.com/operator/2.3-beta/best-practices.html#ulimits   Here are the changes -  right below the first paragraph, add this caution [CAUTION]   ====   Red Hat OpenShift 4.0 and above sets the default limit of the maximum number of process IDs in the cgroup to be 1024; you can read about this here https://access.redhat.com/solutions/5366631 (Red Hat subscription required). A workaround is to increase the pids_limit in both the KubeletConfig and ContainerRuntimeConfig.   ====   see this commit for reference - https://github.com/couchbase/couchbase-operator/pull/29/commits/689a40ed9273a5172528e9b7ea169aae9ccd8ff8  
          anil Anil Kumar (Inactive) made changes -
          Assignee Anil Kumar [ anil ] Maria Shodunke [ JIRAUSER25225 ]
          maria.shodunke Maria Shodunke added a comment - - edited

          Hi Anil Kumar , I don't seem to have access to this repo?

          I know on Github we have: https://github.com/couchbase/couchbase-operator
          From what I understand any changes need to be submitted in Gerrit?

          I can't find the couchbase-operator repo on Gerrit though...

          maria.shodunke Maria Shodunke added a comment - - edited Hi Anil Kumar , I don't seem to have access to this repo? I know on Github we have: https://github.com/couchbase/couchbase-operator From what I understand any changes need to be submitted in Gerrit? I can't find the couchbase-operator repo on Gerrit though...
          anil Anil Kumar (Inactive) added a comment -

          Matt Ingenthron Simon Murray Wayne Siu can someone give Maria Shodunke access to the Gerrit. Thanks! 

          anil Anil Kumar (Inactive) added a comment - Matt Ingenthron Simon Murray Wayne Siu can someone give Maria Shodunke access to the Gerrit. Thanks! 
          ingenthr Matt Ingenthron added a comment -

          I did this back on March 28 when Maria asked me via IM. Maria Shodunke, if you still have trouble, please contact the build team.

          ingenthr Matt Ingenthron added a comment - I did this back on March 28 when Maria asked me via IM. Maria Shodunke , if you still have trouble, please contact the build team.
          maria.shodunke Maria Shodunke added a comment -

          Yep, I have access now and no issues so far! Sorry should have mentioned on here, thanks again Matt Ingenthron .

          maria.shodunke Maria Shodunke added a comment - Yep, I have access now and no issues so far! Sorry should have mentioned on here, thanks again Matt Ingenthron .
          maria.shodunke Maria Shodunke added a comment -

          Change available for review here: https://review.couchbase.org/c/couchbase-operator/+/173468 
          Obviously, we are not merging any docs for now, but this is ready.

          maria.shodunke Maria Shodunke added a comment - Change available for review here: https://review.couchbase.org/c/couchbase-operator/+/173468   Obviously, we are not merging any docs for now, but this is ready.
          maria.shodunke Maria Shodunke made changes -
          Resolution Resolved [ 10400 ]
          Status Open [ 1 ] Resolved [ 5 ]
          build-team Couchbase Build Team added a comment -

          Build couchbase-operator-2.3.1-105 contains couchbase-operator commit 5552c6c with commit message:
          K8S-2504 - Link out to OCP MachineConfig docs

          build-team Couchbase Build Team added a comment - Build couchbase-operator-2.3.1-105 contains couchbase-operator commit 5552c6c with commit message: K8S-2504 - Link out to OCP MachineConfig docs

          People

            maria.shodunke Maria Shodunke
            patrick.stephens Patrick Stephens (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty