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-24652] Ambari metrics collector failed to show data for custo… #18

Closed
wants to merge 3 commits into from

Conversation

Projects
None yet
2 participants
@apappu
Copy link
Contributor

commented Mar 26, 2019

…m metrics (apappu)

What changes were proposed in this pull request?

In Fetch API call , appid is being converted into lower case , have removed that statement.
(Please fill in changes proposed in this fix)

(Please fill in changes proposed in this fix)

How was this patch tested?

Deployed the changes in running AMS server and made API calls with upper case appid and lower case appid - it returned the correct response.
deployed the fix and validated few screens in Grafana and Ambari - it worked fine.

(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.

@asfgit

This comment has been minimized.

Copy link

commented Mar 26, 2019

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

@asfgit

This comment has been minimized.

Copy link

commented Mar 26, 2019

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

@asfgit

This comment has been minimized.

Copy link

commented Mar 26, 2019

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

@apappu apappu changed the base branch from master to trunk Mar 26, 2019

@asfgit

This comment has been minimized.

Copy link

commented Mar 26, 2019

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

@apappu apappu closed this Mar 26, 2019

@apappu apappu deleted the apappu:AMBARI-24652 branch Mar 26, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.