ci: Fix ci_native_fuzz_msan CONTAINER_NAME #26843

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2301-ci-fix-name-msan-🕒 changing 1 files +1 −1

maflcko commented at 10:49 AM on January 7, 2023: member

This avoids a duplicate name with the other msan task, which will lead to errors when running locally:

Error: creating container storage: the container name "ci_native_msan" is already in use by 77350e26f9c36abbb601140cd0b485ead093ff118803c720ca8b10f6bdfa37d2. You have to remove that container to be able to reuse that name: that name is already in use
ci: Fix ci_native_fuzz_msan CONTAINER_NAME fa4e98c77f
DrahtBot commented at 10:49 AM on January 7, 2023: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK fanquake, hebasto

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
DrahtBot added the label Tests on Jan 7, 2023
fanquake commented at 11:21 AM on January 7, 2023: member

We should probably get around to auto-cleaning up containers after a clean exit.
fanquake approved
fanquake commented at 11:22 AM on January 7, 2023: member

ACK fa4e98c77f8c6b693b88c1a6bb811650a6267fdb
hebasto approved
hebasto commented at 12:13 PM on January 8, 2023: member

ACK fa4e98c77f8c6b693b88c1a6bb811650a6267fdb, I've verified that there are no other duplicated CONTAINER_NAME's values.
maflcko merged this on Jan 9, 2023
maflcko closed this on Jan 9, 2023
maflcko deleted the branch on Jan 9, 2023
sidhujag referenced this in commit 30ea9684e2 on Jan 9, 2023
maflcko referenced this in commit f4ef856375 on Jan 12, 2023
bitcoin locked this on Jan 9, 2024

Type	Reviewers
ACK	fanquake, hebasto

Contributors

Labels

Linked (view graph)

#26850 ci: Stop and remove CI container

ci: Fix ci_native_fuzz_msan CONTAINER_NAME #26843

Reviews