Server couldn't normally shutdown (sudo shutdown -hP now) after changing docker runtime dependency

Bug #2020610 reported by Zhixiong Chi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Fix Released
Low
Zhixiong Chi

Bug Description

Brief Description
-----------------
The node couldn't shutdown gracefully after the command 'shutdown -hp now'

Severity
--------
Minor

Steps to Reproduce
------------------
$cat ceph-fuse.yaml
 kind: PersistentVolumeClaim
 apiVersion: v1
 metadata:
   name: rwx-test-claim
 spec:
   accessModes:
   - ReadWriteMany
   resources:
     requests:
       storage: 10Gi
   storageClassName: cephfs
 ---
 apiVersion: apps/v1
 kind: Deployment
 metadata:
   name: wrx-centos
   namespace: default
 spec:
   progressDeadlineSeconds: 600
   replicas: 1
   selector:
     matchLabels:
       run: centos
   template:
     metadata:
       labels:
         run: centos
     spec:
       affinity:
         podAntiAffinity:
           requiredDuringSchedulingIgnoredDuringExecution:
           - labelSelector:
               matchExpressions:
               - key: run
                 operator: In
                 values:
                 - centos
             topologyKey: kubernetes.io/hostname
       containers:
       - name: centos
         image: centos/tools
         imagePullPolicy: IfNotPresent
         command: [ "/bin/bash", "-c", "--" ]
         args: [ "while true; do dd if=/dev/zero of=/mnt1/aaa bs=1K count=100 && sleep 1; done;" ]
         volumeMounts:
         - name: pvc1
           mountPath: "/mnt1"
       restartPolicy: Always
       volumes:
       - name: pvc1
         persistentVolumeClaim:
           claimName: rwx-test-claim

$kubectl create -f ceph-fuse.yaml
$ sudo shutdown -hP now

Expected Behavior
------------------
Shutdown works well.

Actual Behavior
----------------
Reboot.

Reproducibility
---------------
Reproducible

System Configuration
--------------------

Branch/Pull Time/Commit
-----------------------

Last Pass
---------

Timestamp/Logs
--------------

Test Activity
-------------

Workaround
----------

Move the config file starlingx-docker-override.conf from /usr/lib/systemd/system/docker.service.d/starlingx-docker-override.conf to /usr/lib/systemd/system/containerd.service.d/

Changed in starlingx:
assignee: nobody → Zhixiong Chi (zhixiongchi)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to integ (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/starlingx/integ/+/884648

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to integ (master)

Reviewed: https://review.opendev.org/c/starlingx/integ/+/884648
Committed: https://opendev.org/starlingx/integ/commit/8ee62627203c46ffc6a29c5b8cbd93688413a195
Submitter: "Zuul (22348)"
Branch: master

commit 8ee62627203c46ffc6a29c5b8cbd93688413a195
Author: Zhixiong Chi <email address hidden>
Date: Thu May 25 20:41:41 2023 +0800

    ceph: Adjust the override config file to the containerd runtime

    As the current ceph depends on the containerd docker runtime, so
    we move the original config file from docker service to containerd.
    Make sure the correct dependency order and avoid the failure to
    shutdown.
    Meanwhile rename the config script to make it more clearer.

    TestPlan:
    PASS: build-pkgs -a
    PASS: build-image
    PASS: Jenkins Installation
    PASS: Run the testcase pods
    PASS: shutdown -hP now

    Closes-Bug: 2020610

    Signed-off-by: Zhixiong Chi <email address hidden>
    Change-Id: I84719699bdc245cc6f3c0eb6ee4b81544d35459d

Changed in starlingx:
status: In Progress → Fix Released
Ghada Khalil (gkhalil)
Changed in starlingx:
importance: Undecided → Low
tags: added: stx.9.0 stx.distro.other
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.