test with, and support, auth framework specifically.#776
Draft
d-w-moore wants to merge 2 commits intoirods:mainfrom
Draft
test with, and support, auth framework specifically.#776d-w-moore wants to merge 2 commits intoirods:mainfrom
d-w-moore wants to merge 2 commits intoirods:mainfrom
Conversation
Collaborator
Author
|
@d-w-moore Reminder - rebase to |
d-w-moore
force-pushed
the
775.m
branch
2 times, most recently
from
8513424 to
580f23a
Compare
…eation] [_697] Github workflows for running PRC tests Co-authored-by: Terrell Russell <[email protected]> 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
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR will likely include tests in near future that specifically target the
pam_passwordandpam_interactiveschemes.We want to