-
Notifications
You must be signed in to change notification settings - Fork 295
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
kubeflow-pipelines-visualization-server/2.4.0 package update #39775
Conversation
octo-sts
bot
commented
Jan 16, 2025
Please use 👍 or 👎 on this comment to indicate if you agree or disagree with the recommendation. To provide more detailed feedback please comment on the recommendation prefixed with /ai-verify: e.g. /ai-verify partially helpful but I also added bash to the build environment Gen AI suggestions to solve the build error: Based on the error output, I'll analyze and provide specific recommendations: • Detected Error:
• Error Category: Build Configuration/Patch Failure • Failure Point: Patch application step in the pipeline • Root Cause Analysis:
• Suggested Fix:
cd /tmp
git clone https://github.com/kubeflow/pipelines
cd pipelines
git checkout 2.4.0
# Make your dependency updates
git diff > 0001-Bump-dependencies.patch
- runs: |
cd ${{targets.destdir}}/usr/share/app/
# Direct dependency updates
sed -i 's/^tensorflow==.*$/tensorflow>=2.13.0,<3.0.0/' requirements.txt
sed -i 's/^numpy==.*$/numpy>=1.24.0/' requirements.txt
# Add other direct replacements as needed • Explanation: • Additional Notes:
• References: |
Hm, for some reason, recreation of these patches is not working. When the patches don’t apply cleanly, We updated the pip-compile --output-file=requirements.txt --resolver=backtracking requirements.in However, it seems like
I am encountering this error without making any changes to the See full ERROR:
|
I've got a working 2.4.0 build in OddBloke:oddbloke/kubeflow-pipelines-visualization-server-2.4.0, but I based that on #39639 so we're blocked until that lands. |
5adcb95
to
8740d25
Compare
34056ca
to
0dfbef6
Compare
The Jupyter version that upstream now depend on no longer ships (or depends upon packages which ship) a number of scripts, so drop them from the tests.
I don't believe it's required, it was just in the package when these automated tests were generated.
0dfbef6
to
372194a
Compare