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

AMBARI-25034 Ambari is not updating core-site : ha.zookeeper.quorumonce a zookeeper server is added or removed (asnaik) #2910

Merged
merged 1 commit into from
Apr 4, 2019

Conversation

Akhilsnaik
Copy link

AMBARI-25034 Ambari is not updating core-site : ha.zookeeper.quorumonce a zookeeper server is added or removed (asnaik)

What changes were proposed in this pull request?

Tested the UI as per defect description

UT Test successful in the local run

22064 passing (34s)
48 pending

JQMIGRATE: jQuery.support.boxModel is deprecated

JQMIGRATE: jQuery.event.handle is undocumented and deprecated

[INFO]
[INFO] --- apache-rat-plugin:0.12:check (default) @ ambari-web ---
[INFO] RAT will not execute since it is configured to be skipped via system property 'rat.skip'.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:06 min
[INFO] Finished at: 2018-12-11T23:33:17+05:30
[INFO] Final Memory: 17M/225M
[INFO] ------------------------------------------------------------------------

How was this patch tested?

(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

Please review Ambari Contributing Guide before opening a pull request.

…ce a zookeeper server is added or removed (asnaik)
@Akhilsnaik
Copy link
Author

hi @atkach , @jgolieb ,
Please help me review this PR : this is similar to : #2715

@asfgit
Copy link

asfgit commented Apr 3, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5095/
Test FAILed.
Test FAILured.

@ishanbha
Copy link
Contributor

ishanbha commented Apr 3, 2019

@Akhilsnaik For same patches going to different branches you don't need +1 again if it is approved for merge into any other branch. Just add cherry-pick label to it.

@Akhilsnaik
Copy link
Author

retest this please

@asfgit
Copy link

asfgit commented Apr 4, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5099/
Test PASSed.

@Akhilsnaik Akhilsnaik merged commit c94263b into apache:branch-2.7 Apr 4, 2019
@Akhilsnaik Akhilsnaik deleted the AMBARI-25034-2.7 branch April 4, 2019 04:04
payert pushed a commit to payert/ambari that referenced this pull request Apr 16, 2020
…ce a zookeeper server is added or removed (asnaik) (apache#2910)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants