Skip to content

Commit

Permalink
fix: resolverevision endpoint now works correctly for oci
Browse files Browse the repository at this point in the history
Signed-off-by: Blake Pettersson <blake.pettersson@gmail.com>
  • Loading branch information
blakepettersson committed Aug 22, 2024
1 parent 6a27a6d commit 8378bee
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions reposerver/repository/repository.go
Original file line number Diff line number Diff line change
Expand Up @@ -2636,6 +2636,18 @@ func (s *Service) ResolveRevision(ctx context.Context, q *apiclient.ResolveRevis
ambiguousRevision := q.AmbiguousRevision
var revision string
source := app.Spec.GetSourcePtrByIndex(int(q.SourceIndex))

if source.IsOci() {
_, revision, err := s.newOciClientResolveRevision(ctx, repo, ambiguousRevision, true)
if err != nil {
return &apiclient.ResolveRevisionResponse{Revision: "", AmbiguousRevision: ""}, err
}
return &apiclient.ResolveRevisionResponse{
Revision: revision,
AmbiguousRevision: fmt.Sprintf("%v (%v)", ambiguousRevision, revision),
}, nil
}

if source.IsHelm() {
_, revision, err := s.newHelmClientResolveRevision(repo, ambiguousRevision, source.Chart, true)
if err != nil {
Expand Down

0 comments on commit 8378bee

Please sign in to comment.