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

BIGTOP-4081: Add knox support #1247

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JiaLiangC
Copy link
Contributor

@JiaLiangC JiaLiangC commented Apr 7, 2024

Description of PR

Thank you very much, cheelio for working on this #1029
99% of the work on this PR was contributed by cheelio. Due to cheelio's long absence of response and inactivity on this PR, it has been resubmitted in the hope of continuing to drive this matter forward. Based on the original PR, support for pkg suffix has been added, and the version has been updated to 2.0, along with some fixes for 2.0.

The significance of this effort lies in its ability to enhance Knox functionality, particularly in clusters employing Kerberos. Knox's proxy access capabilities provide an intuitive means for users to access the UI interface of Kerberos-enabled clusters.

How was this patch tested?

smoke test and manual test
tested on rocky8
./docker-hadoop.sh -d --create 1 --image bigtop/puppet:trunk-rockylinux-8 --docker-compose-plugin --memory 8g --repo file:///bigtop-home/output --disable-gpg-check --stack knox --smoke-tests knox
image

tested on debian11
./docker-hadoop.sh -d -dcp -C config_debian-11.yaml -F docker-compose.yml -G -L -k knox -s knox -c 1
image

image

tested on ambari deploy
. /etc/profile.d/bigtop.sh;./gradlew knox-clean knox-pkg -PparentDir=/usr/bigtop -PpkgSuffix -PbuildThreads=2C repo
image

image

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
  • Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

@JiaLiangC JiaLiangC force-pushed the BIGTOP-4081 branch 2 times, most recently from bbeebdc to 6be5bdf Compare April 7, 2024 08:10
@JiaLiangC
Copy link
Contributor Author

JiaLiangC commented Apr 7, 2024

@iwasakims @guyuqi @kevinw66 Could you help review this pr?

@kevinw66 kevinw66 self-requested a review April 8, 2024 06:00
@JiaLiangC
Copy link
Contributor Author

@iwasakims @guyuqi @kevinw66 Could you help review this pr?

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.

1 participant