Skip to content

Update apt and brew package lists, ensure latest versions - #245

Merged
bubacoder merged 1 commit into
mainfrom
feature/ansible-brew
Dec 10, 2025
Merged

Update apt and brew package lists, ensure latest versions#245
bubacoder merged 1 commit into
mainfrom
feature/ansible-brew

Conversation

@bubacoder

@bubacoder bubacoder commented Dec 10, 2025

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Added development tools: JSON/YAML query utilities and Gitea CLI for improved developer productivity.
    • Added GitHub Actions workflow testing support.
  • Chores

    • Updated package installation to always deploy latest available versions.
    • Updated host network configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai

coderabbitai Bot commented Dec 10, 2025

Copy link
Copy Markdown
Contributor

Walkthrough

Ansible configuration updates across inventory and role definitions: added development tool packages (jq, yq, tea, act) to Debian environments, updated a host IP address in the inventory, and changed package installation strategy from presence verification to latest-version installation across Homebrew tasks.

Changes

Cohort / File(s) Summary
Inventory configuration
ansible/inventory/group_vars/debian/vars.yaml, ansible/inventory/inventory.yaml
Added jq, yq, and tea (Gitea CLI) to dev_apt_packages; added act to dev_brew_packages. Updated ansible_host for hive group from 192.168.1.243 to 192.168.1.40.
Homebrew task updates
ansible/roles/debian_developer/tasks/20-homebrew.yaml, ansible/roles/mac_base/tasks/10-homebrew.yaml
Changed package installation state from present to latest in four tasks across both roles to ensure latest versions are installed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Package list additions are straightforward and consistent
  • IP address change is a single configuration value update requiring verification of the new address
  • State changes (present → latest) are uniform across tasks; verify intentionality of automatic upgrades versus fixed versioning

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: updating apt and brew package lists and modifying state to ensure latest versions are installed.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/ansible-brew

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1b3a62d and ca32c03.

📒 Files selected for processing (4)
  • ansible/inventory/group_vars/debian/vars.yaml (1 hunks)
  • ansible/inventory/inventory.yaml (1 hunks)
  • ansible/roles/debian_developer/tasks/20-homebrew.yaml (2 hunks)
  • ansible/roles/mac_base/tasks/10-homebrew.yaml (1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
**/*.{yaml,yml}

📄 CodeRabbit inference engine (AGENTS.md)

YAML files must pass YAML linting and validation

Files:

  • ansible/roles/mac_base/tasks/10-homebrew.yaml
  • ansible/inventory/inventory.yaml
  • ansible/roles/debian_developer/tasks/20-homebrew.yaml
  • ansible/inventory/group_vars/debian/vars.yaml
ansible/**/*.{yaml,yml}

📄 CodeRabbit inference engine (AGENTS.md)

Ansible playbooks and roles must pass Ansible linting

Files:

  • ansible/roles/mac_base/tasks/10-homebrew.yaml
  • ansible/inventory/inventory.yaml
  • ansible/roles/debian_developer/tasks/20-homebrew.yaml
  • ansible/inventory/group_vars/debian/vars.yaml
🧠 Learnings (2)
📓 Common learnings
Learnt from: CR
Repo: bubacoder/infra PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-26T20:10:59.672Z
Learning: Use Ansible for infrastructure configuration management
Learnt from: CR
Repo: bubacoder/infra PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-26T20:10:59.672Z
Learning: Applies to ansible/**/*.{yaml,yml} : Ansible playbooks and roles must pass Ansible linting
Learnt from: bubacoder
Repo: bubacoder/infra PR: 89
File: ansible/bootstrap-ansible.sh:23-24
Timestamp: 2025-01-25T11:16:21.899Z
Learning: In the bootstrap-ansible.sh script, the `--ignore-installed` flag is only needed for pip installations on Ubuntu to fix PyYAML conflicts. The Debian installation works fine without this flag.
📚 Learning: 2025-11-26T20:10:59.672Z
Learnt from: CR
Repo: bubacoder/infra PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-26T20:10:59.672Z
Learning: Use Ansible for infrastructure configuration management

Applied to files:

  • ansible/inventory/inventory.yaml
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: build-test-push (ubuntu, ubuntu:24.04)
  • GitHub Check: build-test-push (debian, debian:trixie-slim)
  • GitHub Check: check
🔇 Additional comments (4)
ansible/inventory/inventory.yaml (1)

18-18: LGTM. The IP address update for the hive host is straightforward and syntactically correct.

ansible/inventory/group_vars/debian/vars.yaml (1)

51-53: LGTM. Package list additions are well-formatted, use consistent indentation, and include helpful comments. All four new packages (jq, yq, tea, act) are appropriate for development environments.

Also applies to: 59-59

ansible/roles/mac_base/tasks/10-homebrew.yaml (1)

32-32: Verify the operational impact of switching to state: latest. The change from state: present to state: latest means Homebrew packages will be upgraded to the latest available version on every Ansible run. This could introduce breaking changes or unexpected behavior if package upgrades have incompatible API changes. Ensure this automatic-upgrade strategy is intentional for your environment and consider documenting the rationale, especially if this applies to production-like systems. If version pinning is desired, consider capturing specific versions in separate variables.

Also applies to: 38-38

ansible/roles/debian_developer/tasks/20-homebrew.yaml (1)

6-6: Verify the operational impact of switching to state: latest. Like the mac_base role (File 3), this change enables automatic package upgrades on every Ansible run. Ensure this is intentional and appropriate for your development environment. The changes are consistent across both roles, which suggests a deliberate strategy shift—please confirm this aligns with your infrastructure goals and document the rationale if needed.

Also applies to: 16-16


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bubacoder
bubacoder merged commit f3a7eba into main Dec 10, 2025
6 checks passed
@bubacoder
bubacoder deleted the feature/ansible-brew branch December 10, 2025 16:49
@coderabbitai coderabbitai Bot mentioned this pull request Jun 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant