Skip to content

Fix CI - #313

Merged
charliemirabile merged 3 commits into
masterfrom
fix_ci
Sep 16, 2026
Merged

Fix CI#313
charliemirabile merged 3 commits into
masterfrom
fix_ci

Conversation

@theyoyojo

Copy link
Copy Markdown
Contributor

CI is broken due to changes in default podman-compose logs output format.

In this PR, we this and other minor issues with the test container to rectify pipeline failures.

@charliemirabile charliemirabile left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it work if you just keep only the last two commits (fixing log args and pinning version)? I feel like the others may have been things that were worth trying but not ultimately required to solve the problem.

Comment thread start.sh
Comment thread .containerignore
Comment thread start.sh Outdated
…t container

Signed-off-by: Joel Savitz <joel@underground.software>
podman-compose 1.6.0 introduced formatted prefixes and color options as the
default output for the `podman-compose logs` command [0].

We use sed commands to scan for particular plaintext strings in the logs
output to verify container progress to a certain checkmark, so modify
all these log output invokations with the flags necessary to ensure
plaintext output as before.

[0] https://github.com/containers/podman-compose/releases/tag/v1.6.0

Signed-off-by: Joel Savitz <joel@underground.software>
Signed-off-by: Joel Savitz <joel@underground.software>

@charliemirabile charliemirabile left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@charliemirabile
charliemirabile merged commit aa4442c into master Sep 16, 2026
1 check passed
@charliemirabile
charliemirabile deleted the fix_ci branch September 16, 2026 20:05
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