Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement ORAS helper command #114

Closed
wants to merge 10 commits into from
Closed

Implement ORAS helper command #114

wants to merge 10 commits into from

Conversation

qmuntal
Copy link
Member

@qmuntal qmuntal commented Apr 5, 2024

azoras is a tool that helps work with the ORAS CLI and Azure ACR.
The subcommands implement common workflows for image annotations and maintenance.

It currently implement three basic workflows:

  • azoras install: Install the ORAS CLI from GitHub releases
  • azoras login: Logs in the ORAS CLI to Azure ACR using the Azure CLI
  • azoras deprecate: Adds a end-of-life annotation to the given image or list of images

@qmuntal qmuntal requested review from dagood and gdams April 5, 2024 07:08
@qmuntal qmuntal requested a review from a team as a code owner April 5, 2024 07:08
cmd/azoras/install.go Outdated Show resolved Hide resolved
cmd/azoras/install.go Outdated Show resolved Hide resolved
cmd/azoras/install.go Outdated Show resolved Hide resolved
cmd/azoras/install.go Outdated Show resolved Hide resolved
Comment on lines 20 to 21
azoras doesn't handle authentication, you need to login to your Azure ACR registry with the ORAS CLI.
See https://oras.land/docs/how_to_guides/authentication for more information.
Copy link
Member

Choose a reason for hiding this comment

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

azoras doesn't handle authentication seems to conflict with azoras provides a helper subcommand to login. I think I get what this is trying to say (it won't use any ambient auth if you go directly to run the azoras deprecate command) but that doesn't seem all that important vs. listing the relatively painless commands that you actually need to run to get it set up.

I think this could be moved into a note after the instructions, and I don't think it needs to be worded like azoras is lacking a major feature, just pointing out what's going on behind the scenes.

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point, I'll rephrase this section.

cmd/azoras/deprecate.go Outdated Show resolved Hide resolved
cmd/azoras/azoras.go Show resolved Hide resolved
internal/archive/tar.go Outdated Show resolved Hide resolved
internal/archive/tar.go Show resolved Hide resolved
internal/archive/tar.go Show resolved Hide resolved
@dagood
Copy link
Member

dagood commented Apr 30, 2024

Moved to our internal repo for now while we work on a pipeline etc.

@dagood dagood closed this Apr 30, 2024
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.

2 participants