Skip to content

Update create-replication-endpoints.md - #690

Merged
Vad1mo merged 2 commits into
goharbor:mainfrom
neginkheirmand:main
Sep 20, 2026
Merged

Vad1mo merged 2 commits into
goharbor:mainfrom
neginkheirmand:main

Conversation

@neginkheirmand

Copy link
Copy Markdown
Contributor

[UPDATE: Fix Typo] - space missing
This is the same fix as PR #689, opened against main as requested.

[UPDATE:  Fix Typo] - space missing

Signed-off-by: Negin Kheirmand <neginkheirmand@gmail.com>
[fix typo]: Missing space 

Signed-off-by: Negin Kheirmand <neginkheirmand@gmail.com>
@neginkheirmand

Copy link
Copy Markdown
Contributor Author

This is the same fix as PR #689 and PR #688 (combined), opened against main as requested.

@bupd bupd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@Vad1mo Vad1mo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Whitespace-only fix, correct. The create-replication-endpoints.md typo is still present on main.

The _index.md hunk is already on main via #699 (byte-identical), so it merges as a no-op. No rebase needed.

Follow-up, not this PR: "storage permission" is not an ECR concept. The adapter needs IAM actions for GetAuthorizationToken, DescribeRepositories and CreateRepository (src/pkg/reg/adapter/awsecr/auth.go:134, adapter.go:227,241).

@Vad1mo
Vad1mo merged commit 8022db8 into goharbor:main Sep 20, 2026
5 checks passed
Vad1mo pushed a commit that referenced this pull request Sep 20, 2026
Backport of the same fix as #690, against release-2.14.0.

"password.The" -> "password. The", "storagepermission" -> "storage
permission".
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.

6 participants