fix(pg-upgrade): fixes a bug initiate.sh's conf parser

[zsolt-p]

• this fixes the bug in how initiate.sh parses the postgresql.conf file by ignoring commented-out lines

Summary by CodeRabbit

• Bug Fixes
  • Improved PostgreSQL upgrade process by enhancing configuration value extraction to properly handle indented configuration parameters, ensuring more reliable upgrades in various server environments.

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

[coderabbitai]

Walkthrough

Refactored shell script parameter extraction logic in the PostgreSQL upgrade initialization script. Replaced cat-then-grep pipelines with direct grep patterns that handle whitespace prefixes, enabling matching of indented configuration lines.

Changes

Cohort / File(s)	Summary
PostgreSQL Upgrade Configuration Extraction ansible/files/admin_api_scripts/pg_upgrade_scripts/initiate.sh	Improved extraction patterns for SHARED_PRELOAD_LIBRARIES and PGDATAOLD parameters by replacing pipeline operations with whitespace-aware grep and sed commands for better handling of indented configuration entries

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Whiskers twitching with delight,
A rabbit hops through grep and sed,
Whitespace patterns now aligned just right,
No more pipes to twist your head!
PostgreSQL config flows so neat,
This upgrade's made complete! 🎉

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description is minimal and lacks required sections from the template (like context, changes, or testing details), though it captures the core fix intent.	Expand the description to include the template sections: context/motivation, detailed changes, and any testing performed. Use the provided template to ensure completeness.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly references fixing a bug in initiate.sh's config parser, which matches the primary change of improving the postgresql.conf parsing logic.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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