Skip to content
This repository was archived by the owner on Feb 16, 2024. It is now read-only.

Conversation

adwk67
Copy link
Member

@adwk67 adwk67 commented Sep 16, 2022

Description

Add an airflow job executing and monitoring a spark-k8s job to the demo.

Review Checklist

  • Code contains useful comments
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

@adwk67 adwk67 self-assigned this Sep 16, 2022
@adwk67 adwk67 marked this pull request as ready for review September 19, 2022 08:34
@adwk67 adwk67 requested a review from sbernauer September 19, 2022 08:34
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Worked fine, awesome!

@sbernauer
Copy link
Member

When i now run in the default Namespace spark_pi_submit throws

HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"namespaces \"test-spark-k8s-cr-def\" not found","reason":"NotFound","details":{"name":"test-spark-k8s-cr-def","kind":"namespaces"},"code":404}

I'm trying something....

@sbernauer
Copy link
Member

Ok, my test didn't work

@adwk67
Copy link
Member Author

adwk67 commented Sep 21, 2022

When i now run in the default Namespace spark_pi_submit throws

HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"namespaces \"test-spark-k8s-cr-def\" not found","reason":"NotFound","details":{"name":"test-spark-k8s-cr-def","kind":"namespaces"},"code":404}

I'm trying something....

This worked for me (see earlier comment & screenshot)

@sbernauer
Copy link
Member

Ok, giving it another try

@adwk67
Copy link
Member Author

adwk67 commented Sep 21, 2022

Now using same namespace as demo:

  • non-default:

image

  • default:

image

Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Nice, thanks a lot!

@adwk67
Copy link
Member Author

adwk67 commented Sep 21, 2022

bors merge

bors bot pushed a commit that referenced this pull request Sep 21, 2022
## Description

Add an airflow job executing and monitoring a spark-k8s job to the demo.
@bors
Copy link

bors bot commented Sep 21, 2022

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Airflow demo with spark-k8s [Merged by Bors] - Airflow demo with spark-k8s Sep 21, 2022
@bors bors bot closed this Sep 21, 2022
@bors bors bot deleted the airflow-demo-spark-k8s branch September 21, 2022 11:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants