Skip to content

Disable MIGraphX composable kernels in Jenkinsfile#2268

Merged
pabloantoniom merged 9 commits intodevelopfrom
pablo-try-disable-docker
Mar 26, 2026
Merged

Disable MIGraphX composable kernels in Jenkinsfile#2268
pabloantoniom merged 9 commits intodevelopfrom
pablo-try-disable-docker

Conversation

@pabloantoniom
Copy link
Copy Markdown
Contributor

@pabloantoniom pabloantoniom commented Mar 4, 2026

Motivation

We are having issues at MIGraphX stage in nightly runs:

https://ml-ci-internal.amd.com/job/MLIR/job/mlir-nightly-all/2332/pipeline-overview/?selected-node=2505

The issue is in docker login:

echo ****
docker login -u **** --password-stdin
Error saving credentials: open /home/jenkins/.docker/config.json3036935522: read-only file system
script returned exit code 1

So this PR removes that (no longer necessary). Even after doing that, the build will still fail:

https://ml-ci-internal.amd.com/job/MLIR/job/mlir/job/PR-2268/3/pipeline-overview/

Because of an error in MIGraphX composable kernels. This PR also disables that

Technical Details

  • Remove docker login
  • Disable MIGraphX composable kernels with -DMIGRAPHX_USE_COMPOSABLEKERNEL=OFF flag.

Test Plan

No new test was added

Test Result

MIGraphX stage should now pass

Submission Checklist

@pabloantoniom pabloantoniom requested a review from causten as a code owner March 4, 2026 13:22
@pabloantoniom pabloantoniom changed the title [DO NOT MERGE] Test if disabling docker login fixes the CI issue Remove docker login and disable MIGraphX composable kernels in Jenkinsfile Mar 10, 2026
@pabloantoniom pabloantoniom changed the title Remove docker login and disable MIGraphX composable kernels in Jenkinsfile Disable MIGraphX composable kernels in Jenkinsfile Mar 25, 2026
Copy link
Copy Markdown
Contributor

@dorde-antic dorde-antic left a comment

Choose a reason for hiding this comment

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

should we merge this PR? So that official nightly run start passing? Or this PR is a temporary fix so that we can run custom nightly runs on it?

@pabloantoniom
Copy link
Copy Markdown
Contributor Author

should we merge this PR? So that official nightly run start passing? Or this PR is a temporary fix so that we can run custom nightly runs on it?

I'm trying to, but I'm having a hard time merging this PR. I keep re-running the CI and then someone merges before me. I also tried running an empty run but I just can't. It still runs the MIGraphX step every time, despite having that stage disabled. I dont know what's wrong

@pabloantoniom pabloantoniom merged commit 99b693e into develop Mar 26, 2026
15 checks passed
@pabloantoniom pabloantoniom deleted the pablo-try-disable-docker branch March 26, 2026 14:10
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.

3 participants