Skip to content

AMBARI-24930. Recommendation configs request was failed with custom config group.#2637

Merged
mpapirkovskyy merged 4 commits intoapache:trunkfrom
mpapirkovskyy:AMBARI-24930-trunk
Nov 27, 2018
Merged

AMBARI-24930. Recommendation configs request was failed with custom config group.#2637
mpapirkovskyy merged 4 commits intoapache:trunkfrom
mpapirkovskyy:AMBARI-24930-trunk

Conversation

@mpapirkovskyy
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

ConfigGroup was made comparable.

How was this patch tested?

Manual check.

@asfgit
Copy link
Copy Markdown

asfgit commented Nov 20, 2018

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

@asfgit
Copy link
Copy Markdown

asfgit commented Nov 20, 2018

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

@mpapirkovskyy mpapirkovskyy requested a review from swagle November 20, 2018 17:44
@asfgit
Copy link
Copy Markdown

asfgit commented Nov 21, 2018

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

@adoroszlai adoroszlai requested a review from benyoka November 21, 2018 13:30
@mpapirkovskyy mpapirkovskyy merged commit eaae0af into apache:trunk Nov 27, 2018
vishalsuvagia pushed a commit to vishalsuvagia/ambari that referenced this pull request Feb 26, 2019
…onfig group. (apache#2637)

* AMBARI-24930. Recommendation configs request was failed with custom config group. (mpapirkovskyy)

* AMBARI-24930. Recommendation configs request was failed with custom config group. (mpapirkovskyy)

* AMBARI-24930. Recommendation configs request was failed with custom config group. (mpapirkovskyy)

* AMBARI-24930. Recommendation configs request was failed with custom config group. (mpapirkovskyy)
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.

3 participants