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
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

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

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
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@d-w-moore d-w-moore

Labels

documentation testing

Milestone

ongoing

Development

Successfully merging this pull request may close these issues.

1 participant

@d-w-moore