KNOX-3243 : Added Ambari 3.0.0 support#1140
KNOX-3243 : Added Ambari 3.0.0 support#1140harshith-21 wants to merge 2 commits intoapache:masterfrom
Conversation
|
@harshith-21 Thank you for your contribution.
|
Test Results7 tests 7 ✅ 1s ⏱️ Results for commit b774173. ♻️ This comment has been updated with latest results. |
|
Hi @hanicz, |
|
@hanicz , Ive added few details in the jira description |
Thanks, @harshith-21 ! As well as a screenshot from the Knox Home page with the Ambari service displayed in your test topology. |
gateway-service-definitions/src/main/resources/services/ambari/3.0.0/service.xml
Show resolved
Hide resolved
gateway-service-definitions/src/main/resources/services/ambariui/3.0.0/service.xml
Show resolved
Hide resolved
gateway-service-definitions/src/main/resources/services/ambariui/3.0.0/service.xml
Outdated
Show resolved
Hide resolved
gateway-service-definitions/src/main/resources/services/ambariws/3.0.0/service.xml
Show resolved
Hide resolved
|
on a side note, the airflow integration seem to be abandoned, can i give it a try too :) |
|
@moresandeep, please review the PR again, have done the suggested changes gentle reminder |
KNOX-3243 : Added Ambari 3.0.0 support
What changes were proposed in this pull request?
Added service and rewrite xmls for ambari 3.0.0
How was this patch tested?
Tested in local ambari 3.0.0 + knox 2.0.0 environment
Integration Tests
(Please add or update integration tests
.github/workflows/testsfor the feature you are adding. If no unit test is added, please explain why. Check out.github/workflows/tests/README.mdfor instructions)UI changes
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Knox Contributing Process before opening a pull request.