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

Fix CI #186

Closed
wants to merge 1 commit into from
Closed

Fix CI #186

wants to merge 1 commit into from

Conversation

ericcurtin
Copy link
Collaborator

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

@ericcurtin
Copy link
Collaborator Author

CI is fixed again

@ericcurtin
Copy link
Collaborator Author

@rhatdan we should probably merge this and rebase other PRs before continuing, the tests stopped running...

@ericcurtin
Copy link
Collaborator Author

ericcurtin commented Sep 25, 2024

This PR appeared to fix CI:

#175

but it actually ended up turning off CI.

I'm hoping:

    if sys.stdout.isatty():
        conman_args += ["-t"]

fixes some of the warnings about "-it"

@ericcurtin
Copy link
Collaborator Author

Might take a while, multiple tests failing

@ericcurtin ericcurtin force-pushed the github-ci branch 2 times, most recently from 339d3c1 to 56d5c15 Compare September 25, 2024 11:09
Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
@ericcurtin
Copy link
Collaborator Author

@rhatdan it might be better if you took this PR over me, some tests are broken, been fighting a few hours...

bats also isn't running on macOS CI, which isn't ideal...

rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@rhatdan rhatdan mentioned this pull request Sep 25, 2024
@rhatdan rhatdan closed this Sep 25, 2024
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
rhatdan pushed a commit to rhatdan/ramalama that referenced this pull request Sep 25, 2024
Replaces: containers#186

Use ubuntu 22.04, maybe we can move to 24.04 also in another PR.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
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