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

Review Dockerfile changes for building odh-dashboard in Konflux #356

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MohammadiIram
Copy link
Member

@MohammadiIram MohammadiIram marked this pull request as draft October 8, 2024 07:22
@andrewballantyne
Copy link

No changes can merge into main downstream -- please reopen this PR against our ODH repo https://github.com/opendatahub-io/odh-dashboard

Copy link

sonarcloud bot commented Oct 10, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
11 Security Hotspots

See analysis details on SonarCloud

@dchourasia
Copy link

@andrewballantyne I have sent an email with details of why we want to keep these files to downstream for 2.16 version, please let me know if you have any queries.

@andrewballantyne
Copy link

Nothing can merge to main -- you'll break the auto merger, no? @dchourasia Everything has to merge upstream first to continue the fast forward effort.

Why are we trying to open up PRs directly to downstream?

@dchourasia
Copy link

@andrewballantyne it's other way round.. auto-merge will fail if we add these files to upstream. Please check the details I have provided on email and Jira both.

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