Uploaded image for project: 'OpenShift Windows Containers'
  1. OpenShift Windows Containers
  2. WINC-738

Kubelet service is described in services ConfigMap

XMLWordPrintable

    • Icon: Story Story
    • Resolution: Done
    • Icon: Major Major
    • WMCO 7.0.0
    • None
    • None
    • WINC - Sprint 224, WINC - Sprint 225, WINC - Sprint 226, WINC - Sprint 227

      Description

      The kubelet service definition in the services ConfigMap will be used by the WICD bootstrap command to initially start it, and its state will be maintained when WICD is running as a Windows service.

      Engineering details

      Within the ConfigMap, the kubelet service have the bootstrap flag set to true, so that the `bootstrap` command will know to start it.
      The ignition file must be parsed within WMCO in order to properly populate the command arguments

      Acceptance Criteria

      • WMCO specifies the expected state of the kubelet service in the Services ConfigMap
      • The kubelet service defined within the ConfigMap is the same as the one created by WMCB

            [WINC-738] Kubelet service is described in services ConfigMap

            GitLab CEE Bot added a comment -

            CPaaS Service Account mentioned this issue in merge request !51 of openshift-winc-midstream / openshift-winc-midstream on branch rhaos-4.12-rhel-8_upstream_8d5d94b5b137e0f04a7d40491e805992:

            Updated US source to: 2300c0b Merge pull request #1311 from openshift-cherrypick-robot/cherry-pick-1258-to-release-4.12

            GitLab CEE Bot added a comment - CPaaS Service Account mentioned this issue in merge request !51 of openshift-winc-midstream / openshift-winc-midstream on branch rhaos-4.12-rhel-8_ upstream _8d5d94b5b137e0f04a7d40491e805992 : Updated US source to: 2300c0b Merge pull request #1311 from openshift-cherrypick-robot/cherry-pick-1258-to-release-4.12

            Jose Luis Franco Arza (Inactive) added a comment -

            Oh, sorry for the misunderstanding. Disregard the rest of the comments rteague@redhat.com . I though that the QE-Tracker did not get created for some reason.

            Jose Luis Franco Arza (Inactive) added a comment - Oh, sorry for the misunderstanding. Disregard the rest of the comments rteague@redhat.com . I though that the QE-Tracker did not get created for some reason.

            Russell Teague added a comment -

            rhn-engineering-jfrancoa, The QE Tracker task gets created when this item is moved to DONE. We are still waiting for the last PR to merge but all of WMCO CI is blocked on a networking issue. sebsoto, Please update this with a reference to the networking issue.

            Russell Teague added a comment - rhn-engineering-jfrancoa , The QE Tracker task gets created when this item is moved to DONE. We are still waiting for the last PR to merge but all of WMCO CI is blocked on a networking issue. sebsoto , Please update this with a reference to the networking issue.

            Jose Luis Franco Arza (Inactive) added a comment -

            rteague@redhat.com I see the qe label added, however no QE-Tracker has been created. Is there any manual trigger needed?

            Jose Luis Franco Arza (Inactive) added a comment - rteague@redhat.com I see the qe label added, however no QE-Tracker has been created. Is there any manual trigger needed?

            (old account) Sebastian Soto added a comment -

            Approved + LGTM'd. Blocked by cluster CI failures

            (old account) Sebastian Soto added a comment - Approved + LGTM'd. Blocked by cluster CI failures

            (old account) Sebastian Soto added a comment -

            This isnt working on AWS CCM. I have the solution to fix it ready, just testing it now across cluster setups.

            (old account) Sebastian Soto added a comment - This isnt working on AWS CCM. I have the solution to fix it ready, just testing it now across cluster setups.

            (old account) Sebastian Soto added a comment -

            All comments but one have been resolved, doing some quick testing to respond to the final comment.

            (old account) Sebastian Soto added a comment - All comments but one have been resolved, doing some quick testing to respond to the final comment.

            (old account) Sebastian Soto added a comment -

            rteague@redhat.com I was not able to start working on this until yesterday, as I was still working on my own sprint work. WINC-740's PR has just merged and is now marked as done, so I've assigned this story to myself. I don't see this being completed in time for end of sprint.

            (old account) Sebastian Soto added a comment - rteague@redhat.com I was not able to start working on this until yesterday, as I was still working on my own sprint work. WINC-740 's PR has just merged and is now marked as done, so I've assigned this story to myself. I don't see this being completed in time for end of sprint.

            Russell Teague added a comment -

            sebsoto, Do you anticipate that this work will be completed this sprint since Mohammad is out?

            Russell Teague added a comment - sebsoto , Do you anticipate that this work will be completed this sprint since Mohammad is out?

            Mohammad Shaikh (Inactive) added a comment -

            Handing off the PR after syncing up with sebsoto, who will be continuing this work. 

            Some future work around watching for changes to ignition was identified during review. Engaged with the MCO team on this.

            Mohammad Shaikh (Inactive) added a comment - Handing off the PR after syncing up with sebsoto , who will be continuing this work.  Some future work around watching for changes to ignition was identified during review. Engaged with the MCO team on this.

              rh-ee-ssoto Sebastian Soto
              rh-ee-ssoto Sebastian Soto
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Created:
                Updated:
                Resolved: