Skip to content

ci : disable ubuntu-rocm - #26969

Merged
ggerganov merged 2 commits into
cisc/ci-disable-rocm-cachefrom
cisc/ci-disable-ubuntu-rocm
Aug 12, 2026
Merged

ci : disable ubuntu-rocm#26969
ggerganov merged 2 commits into
cisc/ci-disable-rocm-cachefrom
cisc/ci-disable-ubuntu-rocm

Conversation

@CISC

@CISC CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member

Overview

Disable Ubuntu ROCm Release as ccache does not work and build takes 2hrs+.

Additional information

See #20446 (reply in thread)

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: Achicha
  • Language disclosure: Kipsigis Kalenjin

@CISC
CISC requested a review from a team as a code owner August 12, 2026 13:26
@CISC

CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

Squash and merge stack?

@ggerganov

ggerganov commented Aug 12, 2026

Copy link
Copy Markdown
Member

Squash and merge stack?

Yes, in this particular case it would be better to squash all of them to have less release runs. Right now the resources are very tight.

@CISC CISC changed the title disable ubuntu-rocm ci : disable ubuntu-rocm Aug 12, 2026
stack merge was automatically disabled August 12, 2026 13:34

Pull Request is not mergeable

@CISC

CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

Hmmm, that failed for some reason.

@ggerganov
ggerganov merged commit 9558fa4 into master Aug 12, 2026
3 checks passed
@CISC
CISC deleted the cisc/ci-disable-ubuntu-rocm branch August 12, 2026 13:42
@ggerganov

Copy link
Copy Markdown
Member

It worked on my end, but it didn't squash them.

@CISC

CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

It worked on my end, but it didn't squash them.

Strange.

@CISC

CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

It worked on my end, but it didn't squash them.

Strange.

Some rights thing? It also says you authored them. :)

@CISC

CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

Only the last commit got CI attached though, so same-same (though even that could have been [no release]).

@ggerganov

Copy link
Copy Markdown
Member

Yes, I think it bugged out - it said that stack merge was disabled, even though giving me the option to do so:

image

@ggerganov

Copy link
Copy Markdown
Member

We can cancel many of the queued releases:

image

@CISC

CISC commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

Yes, I think it bugged out - it said that stack merge was disabled, even though giving me the option to do so:

Yeah, I got that message after it indicated the first one got merged, but obviously didn't.

@github-actions github-actions Bot added the devops improvements to build systems and github actions label Aug 12, 2026
Comment on lines -1323 to -1326
# - name: ccache
# uses: ggml-org/ccache-action@v1.2.21
# with:
# key: release-ubuntu-22.04-rocm-${{ matrix.ROCM_VERSION }}

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.

It looks like it was commented out from #26962.

I guess that originated from #20446 (reply in thread)

But now ROCm for Ubuntu exist at all as a result.

@ggerganov what do we do to bring this back?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We need to figure out why ccache does not work and how to fix it. Disabled for reason outlined in OP.

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.

OK, let me look in a fork. I suspect it's ccache size is too small.

brittlewis12 pushed a commit to brittlewis12/llama.cpp that referenced this pull request Aug 17, 2026
* disable ubuntu-rocm

* link PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops improvements to build systems and github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants