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

NAS-133186 / 25.04 / Fix port delegate test #15245

Merged
merged 2 commits into from
Dec 19, 2024
Merged

NAS-133186 / 25.04 / Fix port delegate test #15245

merged 2 commits into from
Dec 19, 2024

Conversation

mgrimesix
Copy link
Contributor

The port delegate CI test is failing when setting an FTP port.

The problem is previous tests left FTP configured with onlyanonymous enabled. This setting requires a valid path for the anonymous directory. Additional factor playing into the failure is that the new api method does parameter checking in a slightly different order.

The fix: Return onlyanonymous to false before commencing the failing test.

Passing test here: http://jenkins.eng.ixsystems.net:8080/job/tests/job/api_tests/2338/

@mgrimesix mgrimesix requested a review from a team December 19, 2024 22:30
@bugclerk
Copy link
Contributor

@bugclerk bugclerk changed the title Fix port delegate test NAS-133186 / 25.04 / Fix port delegate test Dec 19, 2024
tests/api2/test_port_delegates.py Outdated Show resolved Hide resolved
@mgrimesix mgrimesix merged commit 9cfa6cd into master Dec 19, 2024
2 checks passed
@mgrimesix mgrimesix deleted the NAS-133186 branch December 19, 2024 22:42
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Dec 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants