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

test: update a read-only setting #1547

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mantissahz
Copy link
Contributor

Copy link
Collaborator

@khushboo-rancher khushboo-rancher left a comment

Choose a reason for hiding this comment

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

Test steps look fine, we can consider a small change.

manager/integration/tests/test_settings.py Show resolved Hide resolved
@yangchiu
Copy link
Member

@mantissahz Could you help to confirm if this test case can pass on master-head? Because I tried to run this test case, but it failed with assertion error: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/5308/testReport/junit/tests/test_settings/test_setting_update_readonly/

Thanks!

@mantissahz
Copy link
Contributor Author

@mantissahz Could you help to confirm if this test case can pass on master-head? Because I tried to run this test case, but it failed with an assertion error: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/5308/testReport/junit/tests/test_settings/test_setting_update_readonly/

Sure, and no, it would not pass now because we still allow modifying read-only settings by APIs.
Do we have any method to update the setting by the manifest in our regression test?

@yangchiu
Copy link
Member

Sure, and no, it would not pass now because we still allow modifying read-only settings by APIs. Do we have any method to update the setting by the manifest in our regression test?

Yes, kubectl is available in the test, Does calling kubectl patch to update the setting solve the problem?

@mantissahz
Copy link
Contributor Author

Does calling kubectl patch to update the setting solve the problem?

@yangchiu,
Yes, it should solve the problem by the kubectl patch cmd.

Copy link

mergify bot commented Dec 29, 2023

This pull request is now in conflicts. Could you fix it @mantissahz? 🙏

1 similar comment
Copy link

mergify bot commented Feb 6, 2024

This pull request is now in conflicts. Could you fix it @mantissahz? 🙏

Ref: 5989

Signed-off-by: James Lu <jamesluhz@gmail.com>
Signed-off-by: James Lu <james.lu@suse.com>
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.

3 participants