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

[Backport release-0.42] Log a deprecation message when the monolithic provider is run. #680

Merged
merged 4 commits into from
Mar 21, 2024

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Mar 21, 2024

Description

Backport of #679 to release-0.42.

This PR also:

How this PR has been tested

Via an uptest run here.

- Display a deprecation warning in the Upbound marketplace docs
  for the monolith.
- Update the source repo URL to github.com/crossplane-contrib/provider-upjet-azure.

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
(cherry picked from commit c37e4d8)
Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
(cherry picked from commit ea9731e)
@ulucinar
Copy link
Collaborator

/test-examples="examples/network/virtualnetwork.yaml"

@ulucinar
Copy link
Collaborator

Closing and reopening to trigger the CI pipelines.

@ulucinar ulucinar closed this Mar 21, 2024
@ulucinar ulucinar reopened this Mar 21, 2024
@sergenyalcin
Copy link
Collaborator

The https://github.com/crossplane-contrib/provider-upjet-azure/actions/runs/8371871345 uptest fails because of this issue. Could you please pin the UXP_VERSION to 1.14.6-up.1?

- Bump uptest version to v0.11.1
- Bump the build submodule to commit 75a9fe3a

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
… official-providers-ci repo

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
@ulucinar
Copy link
Collaborator

/test-examples="examples/network/virtualnetwork.yaml"

@ulucinar
Copy link
Collaborator

We need to decrease the linter's memory footprint. A local run of make lint has succeeded.

Copy link
Collaborator

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

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

Thanks @ulucinar LGTM!

@ulucinar ulucinar merged commit c859ec7 into release-0.42 Mar 21, 2024
9 of 10 checks passed
@ulucinar ulucinar deleted the backport-679-to-release-0.42 branch March 21, 2024 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants