[GitHub] [ambari-metrics] apappu commented on issue #19: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
apappu commented on issue #19: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/19#issuecomment-476901990
 
 
   @avijayanhwx Can you please review this change?


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] asfgit commented on issue #19: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
asfgit commented on issue #19: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/19#issuecomment-476899439
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/Ambari-Metrics-Github-PR-Builder/34/
   Test PASSed.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] apappu opened a new pull request #19: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
apappu opened a new pull request #19: [AMBARI-24652] Ambari metrics collector 
failed to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/19
 
 
   …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)
   
   ## 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)
   
   1. Deployed the changes in running AMS server and made API calls with upper 
case appid and lower case appid  - it returned the correct response.
   
   2. deployed the fix and validated few screens in Grafana and Ambari - it 
worked fine.
   
   Please review [Ambari Contributing 
Guide](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) 
before opening a pull request.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] apappu closed pull request #18: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
apappu closed pull request #18: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/18
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/18#issuecomment-476890893
 
 
   
   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.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/18#issuecomment-476879512
 
 
   
   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.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/18#issuecomment-476878602
 
 
   
   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.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
asfgit commented on issue #18: [AMBARI-24652] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/18#issuecomment-476875310
 
 
   
   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.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] apappu opened a new pull request #18: [AMBARI-24652] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
apappu opened a new pull request #18: [AMBARI-24652] Ambari metrics collector 
failed to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/18
 
 
   …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](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) 
before opening a pull request.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] apappu closed pull request #17: [AMBARI-25194] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
apappu closed pull request #17: [AMBARI-25194] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/17
 
 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] asfgit commented on issue #17: [AMBARI-25194] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
asfgit commented on issue #17: [AMBARI-25194] Ambari metrics collector failed 
to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/17#issuecomment-476870358
 
 
   
   Refer to this link for build results (access rights to CI server needed): 
   https://builds.apache.org/job/Ambari-Metrics-Github-PR-Builder/29/
   Test FAILed.
   Test FAILured.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [ambari-metrics] apappu opened a new pull request #17: [AMBARI-25194] Ambari metrics collector failed to show data for custo…

2019-03-26 Thread GitBox
apappu opened a new pull request #17: [AMBARI-25194] Ambari metrics collector 
failed to show data for custo…
URL: https://github.com/apache/ambari-metrics/pull/17
 
 
   …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)
   
   ## How was this patch tested?
   
   1. Deployed the changes in running AMS server and made API calls with upper 
case appid and lower case appid  - it returned the correct response.
   2. 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](https://cwiki.apache.org/confluence/display/AMBARI/How+to+Contribute) 
before opening a pull request.
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services