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

fix: update documentation for support matrix #1370

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

matttrach
Copy link
Collaborator

Update support matrix information.

@matttrach matttrach self-assigned this Jul 15, 2024
@matttrach matttrach requested a review from jiaqiluo July 15, 2024 22:40
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@matttrach matttrach requested a review from jiaqiluo July 16, 2024 05:12
Copy link
Member

@jiaqiluo jiaqiluo left a comment

Choose a reason for hiding this comment

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

LGMT.
I left a few comments but they should not block the merging of the PR.

Comment on lines +102 to +103
the `release/v4` branch with 4.0.0+ is aligned with Rancher 2.8,
and the `master` branch with 5.0.0+ is aligned with Rancher 2.9.
Copy link
Member

Choose a reason for hiding this comment

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

nit: maybe removing the leading spaces?

* We have 'out of band' flexibility and are only tied to releasing a new version of the provider when we get a new 2.x Rancher minor version.

See the [compatibility matrix](docs/compatibility-matrix.md) for details.

If you are using Terraform to provision clusters on instances of Rancher 2.7 and 2.8, you must have a separate configuration in a separate dir for each provider. Otherwise, Terraform will overwrite the `.tfstate` file every time you switch versions.
If you are using Terraform to provision clusters on instances of Rancher 2.7 and 2.8,
you must have a separate configuration in a separate dir for each provider.
Copy link
Member

Choose a reason for hiding this comment

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

nit: maybe removing the leading spaces?

@matttrach matttrach merged commit 4c6ed7c into rancher:master Jul 16, 2024
1 check passed
@matttrach matttrach deleted the update-matrix branch July 16, 2024 16:48
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