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.

Sign up for GitHub

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

Jump to bottom

ansible: rebrand test-osuosl-aix715-ppc64-[1,2] #3108

Merged
merged 1 commit into from
Jun 20, 2023
Merged

ansible: rebrand test-osuosl-aix715-ppc64-[1,2] #3108

merged 1 commit into from
Jun 20, 2023

Conversation

aixtools
Copy link
Contributor

@aixtools aixtools commented Jun 20, 2023 •
edited
Loading

as test-osuosl-aix72-ppc64-[5,6]

Checklist
  • commit message has one of the standard prefixes
  • faq.md updated if appropriate
  • other documentation is changed or added (if applicable)
  • playbook changes run through VPC or QPC (if you have access)
  • VPC/QPC not applicable for this PR
  • for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Haroon-Khel
Haroon-Khel approved these changes Jun 20, 2023
View reviewed changes
Copy link
Contributor

@Haroon-Khel Haroon-Khel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Ansible linter error is unrelated, issue raised here #3104

@Haroon-Khel
Copy link
Contributor

@steelhead31 Do the hostnames need to be updated in nagios?

@steelhead31
Copy link
Contributor

@steelhead31 Do the hostnames need to be updated in nagios?

Yes, I'll do it shortly, as I'm just working on a couple of other bits, so I'll do it at the same time.

steelhead31
steelhead31 approved these changes Jun 20, 2023
View reviewed changes
Copy link
Contributor

@steelhead31 steelhead31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@steelhead31 steelhead31 merged commit 942772b into adoptium:master Jun 20, 2023
5 of 6 checks passed
@aixtools aixtools deleted the aixinv03 branch July 27, 2023 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Haroon-Khel Haroon-Khel Haroon-Khel approved these changes

@steelhead31 steelhead31 steelhead31 approved these changes

Assignees
No one assigned
Labels
ansible os:aix provider:osuosl
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aixtools @Haroon-Khel @steelhead31