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

unixPB: Update Nagios_Plugins install for macOSX #1895

Merged
merged 1 commit into from
Feb 8, 2021

Conversation

Willsparker
Copy link
Contributor

Ref: #1716 (comment)

For some reason, brew doesn't like installing anything as the administrator user via ansible, without the become: yes and become_user: "{{ ansible_user }}" lines - I just copied what was done in the Common role for installing build/test tool packages.

I've also added an additional task for ARM64, to symlink to the plugins for ARM64. As ARM64 MacOSX requires homebrew to be installed in /opt/homebrew/, as opposed to /usr/local/ for x64 , the original symlink task doesn't work.

Checklist
  • commit message has one of the standard prefixes
  • FAQ.md updated if appropriate : N/A
  • other documentation is changed or added (if applicable) : N/A
  • playbook changes run through VPC or QPC (if you have access) : They both skip adoptopenjdk tags, so wouldn't run the role
  • for inventory.yml changes, bastillion/nagios/jenkins updated accordingly : N/A

@Willsparker Willsparker added ansible Nagios Nagios monitoring issues os:macOS labels Feb 5, 2021
@Willsparker Willsparker added this to the February 2021 milestone Feb 5, 2021
@Willsparker Willsparker requested a review from gdams February 5, 2021 08:54
@Willsparker Willsparker self-assigned this Feb 5, 2021
@github-actions
Copy link

github-actions bot commented Feb 5, 2021

yamllint Failed

Show Output
::error file=./ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Nagios_Plugins/tasks/nagios_MacOSX.yml,line=47,col=1::[empty-lines] too many blank lines (1 > 0)

Workflow: Linter, Action: karancodeyamllint-github-action, Lint: .

@Willsparker Willsparker force-pushed the nagios_plugins_macosx branch from 8c037fc to 87af854 Compare February 5, 2021 09:33
@Willsparker Willsparker merged commit 0a4a92b into adoptium:master Feb 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ansible Nagios Nagios monitoring issues os:macOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants