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

Customize Zeppelin user interface #1070

Merged
merged 6 commits into from
Mar 3, 2023

Conversation

stvoutsin
Copy link
Collaborator

I've worked on customizing Zeppelin to provide a Gaia look & feel. This was done by modifying the source code & doing a new build, which means that we also need to modify the scripts to use the new build. Some more information and discussion on the related issue here: #659

I've made this into a draft, as I'd like to have a discussion on the best place to store the build & whether the changes to the config look ok before merging them in.

@stvoutsin stvoutsin marked this pull request as ready for review February 18, 2023 11:12
@stvoutsin
Copy link
Collaborator Author

I've now tested this with a build created from our forked version of Zeppelin, with our modifications applied, as well as an upgrade to the version of Zeppelin (0.10.1-gaia-dmp)

@stvoutsin
Copy link
Collaborator Author

Ok, new commit now sets the version to match your proposal

Copy link
Collaborator

@Zarquan Zarquan left a comment

Choose a reason for hiding this comment

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

Small changes to make it clearer.

deployments/hadoop-yarn/ansible/27-install-zeppelin.yml Outdated Show resolved Hide resolved
deployments/hadoop-yarn/ansible/config/zeppelin.yml Outdated Show resolved Hide resolved
Copy link
Collaborator

@Zarquan Zarquan left a comment

Choose a reason for hiding this comment

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

👍 Looks good.

@Zarquan Zarquan merged commit b323f7a into wfau:master Mar 3, 2023
@stvoutsin stvoutsin deleted the upgrade/customise-zeppelin branch May 22, 2023 12:34
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