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

Update vmware-tanzu/carvel-vendir to carvel.dev/vendir #7515

Merged
merged 3 commits into from
Feb 23, 2024

Conversation

antgamdia
Copy link
Contributor

@antgamdia antgamdia commented Feb 23, 2024

Description of the change

When upgrading deps with dependabot, we are getting errors like:

the module path downloading found in your go.mod doesn't match the actual path carvel.dev/vendir found in the dependency's go.mod.
Updating the module path in your go.mod to carvel.dev/vendir should resolve this issue.

so this MR is to use carvel.dev/vendir instead.

Benefits

No more errors when running dependabot.

Possible drawbacks

N/A

Applicable issues

N/A

Additional information

N/A

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Copy link

netlify bot commented Feb 23, 2024

Deploy Preview for kubeapps-dev canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 3b72a87
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/65d8b611dbc68f000825c473

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Comment on lines +11 to +13
// This is a temporary function to convert from the new vendir version to the old
// vendir version. This can be removed once the new vendir version is released.
// More details at https://github.com/vmware-tanzu/kubeapps/pull/7515
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This PR will be partially reverted once kapp-ctrl also updates its vendir dep.

@antgamdia antgamdia merged commit dc1ab14 into vmware-tanzu:main Feb 23, 2024
39 checks passed
@antgamdia antgamdia deleted the updateVendir branch February 23, 2024 16:09
@antgamdia
Copy link
Contributor Author

I've opened a PR in kapp-ctrl to speed up their transition to the new vendir version: carvel-dev/kapp-controller#1497

antgamdia added a commit to antgamdia/kubeapps that referenced this pull request Mar 25, 2024
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
antgamdia added a commit to antgamdia/kubeapps that referenced this pull request Mar 25, 2024
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants