Conversation
|
Could one of the admins verify this patch, please? |
2 similar comments
|
Could one of the admins verify this patch, please? |
|
Could one of the admins verify this patch, please? |
|
Could someone verify this patch, please? |
|
@ktdreyer @akraitman Please help to review this patch. We'd like to enable integration test on Arm64 architecture, so several different points have been identified in order to make the integration test happen on Arm64. |
|
@rosinL @shrek-github Would be great if we can file a bug in Ceph Tracker. |
roles/common/tasks/nagios.yml
Outdated
| - have_epel.rc == 1 | ||
| - ansible_architecture == "x86_64" | ||
|
|
||
| - name: Install nrpe without epel(aarch64) |
There was a problem hiding this comment.
The title says "EPEL support" (which is not correct, because it's not supporting EPEL, it's getting the package from EPEL), the task title says "without epel" (which should be capitalized for consistency, too), but then the URLs are for mirrors of EPEL. So, 1) is it with or without EPEL? (clarify in the code) and 2) why a huawei mirror to the actual RPMs rather than the usual "install the EPEL repo ('epel-release') package, and then install the packages"?
Signed-off-by: chuqifang <chuqifang@hisilicon.com>
rosinL
force-pushed
the
nagios-aarch64
branch
from
aa17a2d to
ab0620d
Compare
4 participants
Signed-off-by: chuqifang chuqifang@hisilicon.com