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

Use default NFS version in nis_client #9006

Merged
merged 1 commit into from Nov 27, 2019

Conversation

OleksandrOrlov
Copy link
Contributor

@OleksandrOrlov OleksandrOrlov commented Nov 26, 2019

On 15-SP2 "Any (Highest Available)" NFS Version is selected by default.

The commit checks the selected value and keeps it as is.

For the previous product versions, the behavior remains the same, so
keep selecting the version explicitly.

On 15-SP2 "Any (Highest Available)" NFS Version is selected by default.

The commit checks the selected value and keeps it as is.

For the previous product versions, the behavior remains the same, so
keep selecting the version explicitly.

Related ticket: [poo#59142](https://progress.opensuse.org/issues/59142)
@OleksandrOrlov OleksandrOrlov added notready WIP Work in progress labels Nov 26, 2019
@jknphy
Copy link
Contributor

jknphy commented Nov 27, 2019

It happen before SLE-15-SP2 as far as I remember, but I don't see nis_client scheduled for any other product. Other than that, LGTM.

@OleksandrOrlov
Copy link
Contributor Author

OleksandrOrlov commented Nov 27, 2019

@jknphy it is also scheduled for Sle12.

It happen before SLE-15-SP2 as far as I remember

Should then I change the version check to another version?

@OleksandrOrlov OleksandrOrlov removed WIP Work in progress notready labels Nov 27, 2019
@jknphy
Copy link
Contributor

jknphy commented Nov 27, 2019

It's fine, I think it was also there for previous SLE-15 but I haven't found this test suite running on qam groups. Let's merge it.

@jknphy jknphy merged commit ebf8f0d into os-autoinst:master Nov 27, 2019
@OleksandrOrlov OleksandrOrlov deleted the 59142_nis_client branch July 10, 2020 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants