Skip to content
This repository has been archived by the owner on Nov 25, 2024. It is now read-only.

Update download URL of uptest #255

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sergenyalcin
Copy link
Member

Description of your changes

Change the download URL of uptest.

I have:

  • Read and followed Upbound's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR, as appropriate.

How has this code been tested

Tested locally

Signed-off-by: Sergen Yalçın <[email protected]>
Copy link
Contributor

@ulucinar ulucinar left a comment

Choose a reason for hiding this comment

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

Thanks @sergenyalcin, lgtm. Btw, after this PR is merged, build pipelines depending on an uptest version which is not present at the new location will break. Either we make this clear so that users of uptest update the version at least to v0.12.0 when they bump their repo's build submodules version, or if we want to be super kind to them, we could also push the previous releases of uptest to the new location so that upgrades are smoother. I currently assume this repo is mostly used in the Upbound organization but it's also a public repo (which might be a motivation for some extra care, for instance the crossplane/upjet-provider-template has a dependecy).

makelib/k8s_tools.mk Outdated Show resolved Hide resolved
Signed-off-by: Sergen Yalçın <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants