Conversation
Made-with: Cursor
…nore Keep quay .gitignore and PLAN.md local-only via root .gitignore. Made-with: Cursor
…ROSAClusterName merge) Made-with: Cursor
Made-with: Cursor
…/macOS mapfile is bash 4+ only and unavailable in zsh. Add a portable helper that fills _QUAY_AWS_TAG_PAIRS via a read loop and use it throughout. Made-with: Cursor
…d clarify password alignment steps Refine the troubleshooting section for database connection errors, emphasizing the need to align the PostgreSQL role password with the `DB_URI`. Include specific SQL commands for updating the password and instructions for updating the `config.yaml` if changes are made. Made-with: Cursor
Made-with: Cursor
- Add validated_version 4.20; drop redundant validation alert
- Tag database VPC Name as quay-${CLUSTER_NAME}-Storage (§1, §3.1, §3.9)
- §9: rollout restart quay-app and quay-mirror after IRSA annotation
Made-with: Cursor
✅ Deploy Preview for rh-cloud-experts ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
|
Whoever reviews this, please squash these commits before merging. |
kmcolli
requested changes
Mar 31, 2026
Contributor
kmcolli
left a comment
kmcolli
left a comment
There was a problem hiding this comment.
need to update a few things
|
|
||
| ### 1.3 Tag helper functions | ||
|
|
||
| Define helpers once (same shell session as **§2**–**§5**). They read **`QUAY_AWS_TAGS_JSON`** from **§1.2**. |
Contributor
There was a problem hiding this comment.
fix formatting - §2–§5
§1.2
|
|
||
| ```bash | ||
| load_quay_aws_tag_pairs | ||
| aws ec2 create-tags --resources "${RESOURCE_ID}" --region "${AWS_REGION}" --tags "${_QUAY_AWS_TAG_PAIRS[@]}" |
Contributor
There was a problem hiding this comment.
RESOURCE_ID has not been set
| ```bash | ||
| export QUAY_AWS_TAGS_JSON='[ | ||
| {"Key":"cost-center","Value":"CC468"}, | ||
| {"Key":"owner","Value":"kherath@redhat.com"} |
Contributor
There was a problem hiding this comment.
you might abstract your email and our cost center
- §1.2: placeholder cost center and email; note to customize tags - §1.3: clearer wording for sections §2 through §5 - EC2 create-tags example: use VPC_DB from §3.1 instead of undefined RESOURCE_ID Made-with: Cursor
Made-with: Cursor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds Deploy Red Hat Quay on ROSA HCP with AWS S3, RDS, and ElastiCache (CLI) at
content/rosa/quay/index.md.Includes: IRSA/STSS3Storage, RDS + TLS, ElastiCache, VPC peering, operator install, troubleshooting,
validated_version: "4.20", and shell helpers compatible with zsh (nomapfile).How to test
make previewand open the new guide under /experts/rosa/quay/Related
validated_versionguidance.