Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docker artifact test fails with a PR to drop supervisor from the Docker image #10117

Open
syuu1228 opened this issue Feb 18, 2025 · 1 comment
Assignees

Comments

@syuu1228
Copy link
Contributor

I opened scylladb/scylladb#22883 to drop supervisor from the Docker image, but CI test failed because it missing supervisor.
The error message says:

exception=The command '/bin/bash -c apt-get update &&     echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections &&     apt install -y sudo &&     adduser --disabled-password --gecos "" $USER || true  &&     usermod -aG sudo $USER &&     sudo -Hu $USER sh -c "mkdir -m 700 ~/.ssh" &&     sudo -Hu $USER sh -c "echo 'mkdir -p /home/$USER/.cassandra' > ~/.ssh/rc" &&     sudo -Hu $USER sh -c "echo 'sudo test -f /root/.cqlshrc && sudo cp /root/.cqlshrc /home/$USER/.cassandra/cqlshrc' >> ~/.ssh/rc" &&     sudo -Hu $USER sh -c "echo 'sudo test -f /root/.cassandra/cqlshrc && sudo cp /root/.cassandra/cqlshrc /home/$USER/.cassandra/cqlshrc' >> ~/.ssh/rc" &&     echo "$USER  ALL=(ALL)       NOPASSWD: ALL" >> /etc/sudoers &&     apt install --no-install-recommends -y         iproute2         syslog-ng         rsync         docker-ce-cli &&     rm -rf /var/lib/apt/lists/* &&     echo "autostart=false" >> /etc/supervisord.conf.d/scylla-server.conf &&     echo "user=scylla" >> /etc/supervisord.conf.d/scylla-server.conf &&     echo "autostart=false" >> /etc/supervisord.conf.d/scylla-housekeeping.conf &&     echo "user=scylla" >> /etc/supervisord.conf.d/scylla-housekeeping.conf' returned a non-zero code: 1

https://jenkins.scylladb.com/job/releng-testing/job/artifacts/job/artifacts-docker-test/562/consoleFull#537147054fcc21424-66d2-4bd8-8e0d-9746405e5b16

So likely it failed because /etc/supervisord.conf.d is missing.

We need a way to pass the CI to merge the PR.

@fruch
Copy link
Contributor

fruch commented Feb 18, 2025

@syuu1228 the way is to fix what needed in SCT.

why this is been removed ? what is the equivalent of it ?
how manager agent going to work ?

i.e. where is the test plan for that feature ?

@yaronkaikov yaronkaikov assigned syuu1228 and unassigned syuu1228 Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants