Skip to content

test with, and support, auth framework specifically. #776

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.

Sign up for GitHub

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

Jump to bottom
Draft
d-w-moore wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

test with, and support, auth framework specifically. #776

d-w-moore wants to merge 2 commits into from

Conversation

@d-w-moore
Copy link
Collaborator

@d-w-moore d-w-moore commented Oct 24, 2025 •
edited
Loading

This PR will likely include tests in near future that specifically target the pam_password and pam_interactive schemes.

We want to

  • test a permutation of options when authenticating under the PAM auth-framework protocols just as we did under the legacy auth
  • assert that all the various utility infrastructure works, e.g. that .irodsA generation works under iRODS 5+ .
@d-w-moore d-w-moore marked this pull request as draft October 24, 2025 18:28
@d-w-moore d-w-moore self-assigned this Oct 24, 2025
@d-w-moore d-w-moore added this to the ongoing milestone Oct 24, 2025
@d-w-moore
Copy link
Collaborator Author

d-w-moore commented Oct 24, 2025 •
edited
Loading

@d-w-moore Reminder - rebase to main after #772 is merged (that is, once irods/test/harness/test_script_parameters exists)
(until then, this pull request duplicates a lot of what is in that one)
@d-w-moore d-w-moore force-pushed the 775.m branch 2 times, most recently from 8513424 to 580f23a Compare October 24, 2025 18:44
@d-w-moore
[state of supporting PR irods#772 as of the current Pull Request's cr…
36ad7b5 
…eation] [_697] Github workflows for running PRC tests Co-authored-by: Terrell Russell <tgr@renci.org> Includes a docker compose configuration in which the PRC test suite can be run in a way representative of typical operational use, and the version of iRODS server and python interpreter that we install are also easily reconfigurable. Additionally, this affords us the opportunity to run the test suite on a client node all its own, the iRODS server being reachable only via the network. [_502] test harness and container-based tests A new test harness is introduced in which we construct a new container (using either Docker or podman) for each test program we run. This allows full customization of the container environment for the particular needs of each test. Accordingly, included with the Github workflows (in a separate commit) is a full run of the PRC test suite with the iRODS server and catalog DB server running in the same container as the client. In the process of putting old tests through new rigors, faults were found and corrected in some of those tests. completely remove {send,recv}_oneshot sync mechanism review comments Comments moved to *.yml files. README.md no longer needed. update irods/harness/README deindent correct forgotten comment symbol clearer comment in run_tests.sh suggestions nr. 1 10-22-25 13:19 suggestions nr. 2 10-22-25 smchg package repo chg comments chg
@d-w-moore
[_775] add test for pam_password secrets file generation
4b1f202 
prc_write_irodsA.py invokes a function in the irods.client_init module which adapts to iRODS server version, using the new authentication framework in iRODS 4.3+ and legacy auth in previous versions.
@d-w-moore d-w-moore mentioned this pull request Oct 25, 2025
Merged
@alanking alanking removed this from the ongoing milestone Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation testing

2 participants

@d-w-moore @alanking