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

EZESC-1933 -Update wordcount.yaml to remove duplicate volume #180

Open
wants to merge 1 commit into
base: release/fy24-q2
Choose a base branch
from

Conversation

abe-hpe
Copy link

@abe-hpe abe-hpe commented May 8, 2024

The original example from release/fy24-q2 doesn't work because 'shared-volume' can't mount kubeflow-shared-pv, because the Spark Operator pods already have a volume called 'pv1' mounted to that same PVC. To get the example to work, we have to remove all references to the shared-volume volume and volume mount, and then update mainApplicationFile and 'arguments' to get the necessary data from its location on the 'pv1' mount.

Provide a clear and concise description of the content changes you're proposing. List all the changes you are making
to the content.

  • Updated section ___ with ___.
  • Added new subsection ___ under ___.
  • Removed outdated information from ___.
  • ...

Closes #...

If there is no issue related to this PR, kindly create one first to describe the motivation behind these changes.

Checklist:

  • I have checked that my enhancements are not duplicates of existing content changes or additions.
  • I have tested the changes in a working environment to ensure they function as intended.
  • I have followed the style guide
    outlined in the contribution guidelines.

Reviewer's Tasks (for maintainers reviewing this PR):

  • Verify that the tutorial functions correctly in a live environment.
  • Verify that the updated content aligns with the style guide
    in the contribution guidelines.
  • Check for consistency, grammar, and clarity throughout the updated content.
  • Check that the related GitHub issue is up-to-date.

The original example from release/fy24-q2 doesn't work because 'shared-volume' can't mount kubeflow-shared-pv, because the Spark Operator pods already have a volume called 'pv1' mounted to that same PVC. To get the example to work, we have to remove all references to the shared-volume volume and volume mount, and then update mainApplicationFile and 'arguments' to get the necessary data from its location on the 'pv1' mount.
@mayankgarg-23
Copy link

@abe-hpe can we make similar change for spark-pi example as well otherwise it will also face similar issue.

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.

2 participants