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

issue with lazy sync functionality #319

Closed
Sanrahul02 opened this issue Nov 9, 2023 · 1 comment · Fixed by #322
Closed

issue with lazy sync functionality #319

Sanrahul02 opened this issue Nov 9, 2023 · 1 comment · Fixed by #322
Labels
bug This issue describes a defect or unexpected behavior

Comments

@Sanrahul02
Copy link

Sanrahul02 commented Nov 9, 2023

Background:

The vendir.yml file contains the configuration lazy: true. Upon running vendir sync on a clean project, the project was updated with the vendor directory and all dependencies, along with the vendir.lock.yaml. However, running vendir sync a second time resulted in the complete clearance of the vendor/ directory.

sample vendir.yml :

apiVersion: vendir.k14s.io/v1alpha1
kind: Config
directories:
- path: vendor
  contents:
  - path: dir
    lazy: true
    directory:
      path: ./some-content

What I expected:
I expected the vendir sync to skip populating the vendor directory in the second run without clearing the directory.

Environment:

Vendir version: 0.36.0
Operating System: Photon 3/5 or Ubuntu


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@Sanrahul02 Sanrahul02 added bug This issue describes a defect or unexpected behavior carvel-triage This issue has not yet been reviewed for validity labels Nov 9, 2023
@fritzduchardt
Copy link
Contributor

@Sanrahul02 Thank you for reporting this.

@github-actions github-actions bot removed the carvel-triage This issue has not yet been reviewed for validity label Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue describes a defect or unexpected behavior
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants