Skip to content

Conversation

tripan27
Copy link
Collaborator

@tripan27 tripan27 commented Aug 26, 2025

Description

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content

Run the pipeline

If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.

Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:

/run pipeline

Checklist for reviewers

  • If relevant, a test for the change is included or updated with this PR.
  • If relevant, documentation for the change is included or updated with this PR.

For mergers

  • Use a conventional commit message to set the release level. Follow the guidelines.
  • Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
  • Use the Squash and merge option.

@tripan27
Copy link
Collaborator Author

run /pipeline

@BabithaPadiri BabithaPadiri self-requested a review August 26, 2025 10:02
Copy link
Contributor

@surajsbharadwaj surajsbharadwaj left a comment

Choose a reason for hiding this comment

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

Please have a look at comments.

@tripan27
Copy link
Collaborator Author

tripan27 commented Sep 1, 2025

run /pipeline

Copy link
Contributor

@surajsbharadwaj surajsbharadwaj left a comment

Choose a reason for hiding this comment

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

COMMON dev assets shouldn't be changed. revert it

@surajsbharadwaj surajsbharadwaj changed the title chore: configure AIX services via ansible integration. chore: configure AIX services via ansible integration Sep 1, 2025
Copy link
Contributor

@surajsbharadwaj surajsbharadwaj left a comment

Choose a reason for hiding this comment

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

Too many repeated tags.

  1. You can use block to group tasks and just have one tags for each block. Too many repetitions
  2. Reorder tasks:
    Configure Proxy
    Install curl package
    Extend Rootvg
    Create data filesystem
    Configure NFS
    Configure NTP
    Configure DNS (new task must be added)
  3. On reboot: NFS disk is not mounted. Make the NFS Disk to be persistent on reboot. If the NFS is not reachable boot should proceed and shouldn't hang
  4. On reboot NTP is not started: missing chrctcp -S xntpd # enable xntpd to start at boot
  5. Count is missing for module pi_aix_configure_services

@surajsbharadwaj surajsbharadwaj changed the title chore: configure AIX services via ansible integration fix: configure AIX services via ansible integration Sep 1, 2025
@tripan27
Copy link
Collaborator Author

tripan27 commented Sep 3, 2025

/run pipeline

@tripan27
Copy link
Collaborator Author

tripan27 commented Sep 3, 2025

/run pipeline

Copy link
Contributor

@surajsbharadwaj surajsbharadwaj left a comment

Choose a reason for hiding this comment

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

  1. New DNS task
  2. Include names and move the when condition for the block
  3. DOCS UPDATE , VESION, LAST UPDATED ACROSS ALL FILES

@tripan27 tripan27 merged commit 18af9ee into main Sep 5, 2025
@tripan27 tripan27 deleted the aix-ansible branch September 5, 2025 13:44
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 9.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants