-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
feat: oci support (Beta) #18646
base: master
Are you sure you want to change the base?
feat: oci support (Beta) #18646
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@blakepettersson , The overall code on the PR looks good so far. This is an amazing start to the feature. I have not tested stuff yet though.
One thing we might need to look into is how can we reduce the number of calls to OCI repositories like adding some caching for resolve revision and extract calls. Something what we already do for git ls-remote calls. But this can be a future enhancement.
00a0e52
to
53011c3
Compare
53011c3
to
c682005
Compare
72b5568
to
06c9b58
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #18646 +/- ##
==========================================
- Coverage 55.20% 53.11% -2.09%
==========================================
Files 338 339 +1
Lines 57090 57614 +524
==========================================
- Hits 31516 30604 -912
- Misses 22878 24329 +1451
+ Partials 2696 2681 -15 ☔ View full report in Codecov by Sentry. |
48bf124
to
b4a2d27
Compare
ui/src/app/applications/components/application-status-panel/revision-metadata-panel.tsx
Outdated
Show resolved
Hide resolved
@@ -40,6 +40,7 @@ export interface NewHTTPSRepoParams { | |||
project?: string; | |||
forceHttpBasicAuth?: boolean; | |||
enableOCI: boolean; | |||
insecureHttpOnly: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Around line 308 of this file, it should show an icon for the repo type. Unfortunately, the change should be in the argo ui repo, so will need to create a PR there. See:
https://github.com/argoproj/argo-ui/blob/master/src/styles/argo-icon.scss and the icons folder there, that should contain the new oci svg icon.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@keithchong that's a good point! I'm not much of a designer though, so would need help from someone to create the icon...
message += ' (' + revision.substring(0, 14) + ')'; | ||
} else { | ||
message += ' (' + revision.substring(0, 7) + ')'; | ||
} | ||
} | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible that the <Revision>
field in the lines below need to take into account the format of the revision
value as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possibly - do you have any formatting in mind? Would you want the sha256
prefix to be removed?
a439909
to
2e7848c
Compare
7243332
to
d2963a5
Compare
1409f54
to
2abf6db
Compare
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
If a tar file is embedded in the layer, decompress it and move its contents to the manifests dir. Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
This is really adding a bunch of prereqs to be able to do e2e tests with oci. This installs ORAS CLI, and adds CLI support for OCI repos. Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Co-authored-by: Michael Crenshaw <[email protected]> Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
Signed-off-by: Blake Pettersson <[email protected]>
78be64f
to
3a76a77
Compare
This is a WIP for OCI support in Argo CD. This is the implementation PR for the proposal in #17564.
Checklist: