Skip to content

Conversation

@Denyme24
Copy link
Contributor

@Denyme24 Denyme24 commented Jan 13, 2026

Pull Request Template for Kubeflow Manifests

fixes #3325

✏️ Summary of Changes

Fixed the Spark Operator test script to use the correct kubeflow overlay instead of standalone, and added the --force-conflicts flag to resolve "metadata.annotations: Too long" errors during installation.

Changes:

  • Updated overlay path from overlays/standalone to overlays/kubeflow in tests/spark_install.sh
  • Added --force-conflicts flag to kubectl apply command to handle annotation size limits

📦 Dependencies

List any dependencies or related PRs (e.g., "Depends on #123").

🐛 Related Issues

Link any issues that are resolved or affected by this PR.

✅ Contributor Checklist

  • I have tested these changes with kustomize. See Installation Prerequisites.
  • All commits are signed-off to satisfy the DCO check.
  • I have considered adding my company to the adopters page to support Kubeflow and help the community, since I expect help from the community for my issue (see 1. and 2.).

You can join the CNCF Slack and access our meetings at the Kubeflow Community website. Our channel on the CNCF Slack is here #kubeflow-platform.

@github-actions
Copy link

Welcome to the Kubeflow Manifests Repository

Thanks for opening your first PR. Your contribution means a lot to the Kubeflow community.

Before making more PRs:
Please ensure your PR follows our Contributing Guide.
Please also be aware that many components are synchronizes from upstream via the scripts in /scripts.
So in some cases you have to fix the problem in the upstream repositories first, but you can use a PR against kubeflow/manifests to test the platform integration.

Community Resources:

Thanks again for helping to improve Kubeflow.

@Denyme24 Denyme24 changed the title docs: update Spark operator installation command to include --force-conflicts docs: use server-side apply in README Jan 13, 2026
@Denyme24 Denyme24 force-pushed the docs/update-readme-server-side-apply branch from 138b4e8 to 65a4e9f Compare January 14, 2026 04:43
@juliusvonkohout
Copy link
Member

Thank you
/lgtm
/approve

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: juliusvonkohout

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@juliusvonkohout juliusvonkohout changed the title docs: use server-side apply in README documentation: use server-side apply in README Jan 15, 2026
@google-oss-prow google-oss-prow bot merged commit e33c0bb into kubeflow:master Jan 15, 2026
8 of 9 checks passed
@Denyme24 Denyme24 deleted the docs/update-readme-server-side-apply branch January 15, 2026 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spark-Operator annotations too long

2 participants