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

[Self Cert Tool] Enhance PreFlight checks to get actual count of PID limit on containers.

    XMLWordPrintable

Details

    • Improvement
    • Status: Open
    • Minor
    • Resolution: Unresolved
    • None
    • not-targeted
    • testing
    • None
    • 1

    Description

      The Self Certification tool performs a couple of pre-flight tests to ensure that the underlying system is suitable for the Operator tests to run. 

      These include checking number of processes(ulimit -u) and number of open files(ulimit -n).

      These checks although sufficient doesn't provide an informative picture about the system apart from making sure the values are good enough for tests to run.

      We should either enhance these checks or add new ones to highlight the thread limits of the container (eg: 1024 in Openshift containers) and/or other required parameters upon further discussion.

      Attachments

        Issue Links

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

          Activity

            prateek.kumar Prateek Kumar created issue -

            This ticket would need a discussion within the kubernetes team first before the path to resolution is decided. Hence no fix version is tagged on it. 

             

            prateek.kumar Prateek Kumar added a comment - This ticket would need a discussion within the kubernetes team first before the path to resolution is decided. Hence no fix version is tagged on it.   
            patrick.stephens Patrick Stephens (Inactive) made changes -
            Field Original Value New Value
            Link This issue relates to MB-47591 [ MB-47591 ]
            patrick.stephens Patrick Stephens (Inactive) made changes -
            Link This issue relates to CBSE-10914 [ CBSE-10914 ]

            The intent was to see if we can fork-bomb or otherwise detect the lower thread limits in CBSE-10914. This should also be something server does and is being tracked separately under MB-47591 but if we can indicate a failure then that would be good.

            A test case that just spins up the https://github.com/bostrt/spew-procs container (http://quay.io/harpatil/spew-process) to see how many it gets to: https://bugzilla.redhat.com/show_bug.cgi?id=1844447 

            patrick.stephens Patrick Stephens (Inactive) added a comment - - edited The intent was to see if we can fork-bomb or otherwise detect the lower thread limits in CBSE-10914. This should also be something server does and is being tracked separately under MB-47591 but if we can indicate a failure then that would be good. A test case that just spins up the https://github.com/bostrt/spew-procs  container ( http://quay.io/harpatil/spew-process ) to see how many it gets to: https://bugzilla.redhat.com/show_bug.cgi?id=1844447  
            patrick.stephens Patrick Stephens (Inactive) made changes -
            Description The Self Certification tool performs a couple of pre-flight tests to ensure that the underlying system is suitable for the Operator tests to run. 

            These include checking number of processes(ulimit -u) and number of open files(ulimit -n).

            These checks although sufficient doesn't provide an informative picture about the system apart from making sure the values are good enough for tests to run.

            We should either enhance these checks or add new ones to highlight the thread limits of the container (eg: 1024 in Openshift containers) and/or other required parameters upon further discussion.

            [~patrick.stephens] : Assigning this to you at the moment, Please let me know if it needs to be changed.
            The Self Certification tool performs a couple of pre-flight tests to ensure that the underlying system is suitable for the Operator tests to run. 

            These include checking number of processes(ulimit -u) and number of open files(ulimit -n).

            These checks although sufficient doesn't provide an informative picture about the system apart from making sure the values are good enough for tests to run.

            We should either enhance these checks or add new ones to highlight the thread limits of the container (eg: 1024 in Openshift containers) and/or other required parameters upon further discussion.
            patrick.stephens Patrick Stephens (Inactive) made changes -
            Assignee Patrick Stephens [ JIRAUSER25332 ] Anil Kumar [ anil ]

            Build couchbase-operator-backup-1.1.1-103 contains couchbase-operator-backup commit 38e3ef0 with commit message:
            K8S-2514: Fix Permissions

            build-team Couchbase Build Team added a comment - Build couchbase-operator-backup-1.1.1-103 contains couchbase-operator-backup commit 38e3ef0 with commit message: K8S-2514 : Fix Permissions
            simon.murray Simon Murray made changes -
            Link This issue relates to K8S-2485 [ K8S-2485 ]
            simon.murray Simon Murray added a comment -

            Check 2485 for a potential fix or duplicate

            simon.murray Simon Murray added a comment - Check 2485 for a potential fix or duplicate
            simon.murray Simon Murray made changes -
            Component/s testing [ 14116 ]
            Component/s kubernetes [ 14324 ]
            Component/s openshift [ 14325 ]
            simon.murray Simon Murray made changes -
            Fix Version/s not-targeted [ 16613 ]
            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
            anil Anil Kumar added a comment - - edited

            Simon Murray can you confirm we have added a check in Self Certification Tool to notify users of the thread limits of the container. 

             

            CC Roo Thorp 

            anil Anil Kumar added a comment - - edited Simon Murray can you confirm we have added a check in Self Certification Tool to notify users of the thread limits of the container.    CC Roo Thorp  
            anil Anil Kumar made changes -
            Assignee Anil Kumar [ anil ] Simon Murray [ simon.murray ]
            simon.murray Simon Murray added a comment -

            We added in an rlimit check for PID_MAX.  Nothing beyond that.

            simon.murray Simon Murray added a comment - We added in an rlimit check for PID_MAX.  Nothing beyond that.

            People

              simon.murray Simon Murray
              prateek.kumar Prateek Kumar
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty