Skip to content

Comments

Fix docuhub readme#344

Open
benlee0423 wants to merge 3 commits intomainfrom
fix-docuhub-readme
Open

Fix docuhub readme#344
benlee0423 wants to merge 3 commits intomainfrom
fix-docuhub-readme

Conversation

@benlee0423
Copy link
Contributor

[Short description explaining the high-level reason for the pull request]

Additions

Removals

Updates

Testing

Method

Screenshots / output snippets

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)

Testing checklist

Target Environment support

  • Windows (wsl)
  • Linux
  • MaxOs (apple silicon)

@benlee0423
Copy link
Contributor Author

@arpita0911patel
The build failed because the token used is only authorized to update our Docker build side. It doesn’t have permission to push to the awiciroh/ciroh-ngen-image repository.
To fix: we have to supply another docker hub token that have access to awiciroh/ciroh-ngen-image.

@benlee0423
Copy link
Contributor Author

benlee0423 commented Nov 5, 2025

curl -s https://hub.docker.com/v2/repositories/awiciroh/ciroh-ngen-image/ | grep -E '"user"|"name"|"namespace"'
This returns:
"permissions":{"read":true,"write":false,"admin":false}

token should be created on cirohua team.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants