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

container: cleanup container systemd units #7483

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

clwluvw
Copy link
Member

@clwluvw clwluvw commented Feb 18, 2024

No description provided.

@clwluvw clwluvw force-pushed the container-run branch 3 times, most recently from 0c4db8f to 0a057fa Compare February 18, 2024 12:42
@clwluvw clwluvw changed the title container: cleanup container systemd [Skip CI] container: cleanup container systemd Feb 18, 2024
@clwluvw clwluvw changed the title [Skip CI] container: cleanup container systemd container: cleanup container systemd Feb 18, 2024
@clwluvw
Copy link
Member Author

clwluvw commented Feb 18, 2024

jenkins test centos-container-all_daemons

@clwluvw clwluvw changed the title container: cleanup container systemd [Skip CI] container: cleanup container systemd units Feb 18, 2024
@clwluvw clwluvw changed the title [Skip CI] container: cleanup container systemd units container: cleanup container systemd units Feb 18, 2024
@clwluvw
Copy link
Member Author

clwluvw commented Feb 18, 2024

jenkins test centos-container-all_daemons

@clwluvw clwluvw changed the title container: cleanup container systemd units [Skip CI] container: cleanup container systemd units Feb 18, 2024
@clwluvw clwluvw changed the title [Skip CI] container: cleanup container systemd units container: cleanup container systemd units Feb 18, 2024
@clwluvw
Copy link
Member Author

clwluvw commented Feb 18, 2024

jenkins test centos-container-all_daemons

@clwluvw clwluvw changed the title container: cleanup container systemd units [Skip CI] container: cleanup container systemd units Feb 18, 2024
@clwluvw clwluvw changed the title [Skip CI] container: cleanup container systemd units container: cleanup container systemd units Feb 18, 2024
@clwluvw
Copy link
Member Author

clwluvw commented Feb 18, 2024

jenkins test centos-container-all_daemons

@clwluvw clwluvw changed the title container: cleanup container systemd units [Skip CI] container: cleanup container systemd units Feb 18, 2024
@clwluvw clwluvw changed the title [Skip CI] container: cleanup container systemd units container: cleanup container systemd units Feb 18, 2024
@clwluvw
Copy link
Member Author

clwluvw commented Feb 18, 2024

jenkins test centos-container-all_daemons

@clwluvw clwluvw force-pushed the container-run branch 2 times, most recently from 42d3ea4 to e971e66 Compare February 18, 2024 22:01
tox-cephadm.ini Show resolved Hide resolved
* Make common params of container args in a var to avoid duplication
* The /var/lib/ceph/crash mount was missing after 637ca81
* Add CEPH_USE_RANDOM_NONCE as it's needed when running inside container (can be removed for squid later)
* Add NODE_NAME as some part of ceph code relies on this var
* add default logging opts for

Signed-off-by: Seena Fallah <[email protected]>
Diff option can help in CI when a task fail because of a corrupted render in previous tasks.

Signed-off-by: Seena Fallah <[email protected]>
@guits
Copy link
Collaborator

guits commented Feb 19, 2024

jenkins test centos-container-lvm_osds

@guits guits merged commit 03009f9 into ceph:main Feb 19, 2024
27 checks passed
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

Successfully merging this pull request may close these issues.

2 participants