Skip to content

Commit

Permalink
Add iRODS 4.2.12 to the test matrix
Browse files Browse the repository at this point in the history
Add a struct initialiser for modAVUMetadataInp_t which has had a new
nested struct added in 4.2.12.

Update the local Dockerfile to 4.2.12 and make it similar the the
iRODS development image in the wtsi-npg/containers repo.

Update the docker-compose.yml file to use a 4.2.12 server.
  • Loading branch information
kjsanger committed May 16, 2023
1 parent a3c9c5b commit 32c3e38
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 5 deletions.
5 changes: 5 additions & 0 deletions .github/workflows/unit-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,11 @@ jobs:
build_image: "ghcr.io/wtsi-npg/ub-18.04-irods-clients-dev-4.2.11:latest"
server_image: "ghcr.io/wtsi-npg/ub-18.04-irods-4.2.11:latest"
experimental: false
# iRODS 4.2.12 clients on bionic
- irods: "4.2.12"
build_image: "ghcr.io/wtsi-npg/ub-18.04-irods-clients-dev-4.2.12:latest"
server_image: "ghcr.io/wtsi-npg/ub-18.04-irods-4.2.12:latest"
experimental: false
# iRODS 4.3.0 clients on bionic
- irods: "4.3.0"
build_image: "ghcr.io/wtsi-npg/ub-18.04-irods-clients-dev-4.3.0:latest"
Expand Down
13 changes: 11 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ RUN echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selectio
apt-utils \
ca-certificates \
curl \
dirmngr \
gpg \
gpg-agent \
lsb-release \
Expand All @@ -17,7 +18,7 @@ ENV LANG=en_GB.UTF-8 \
LANGUAGE=en_GB \
LC_ALL=en_GB.UTF-8

ENV IRODS_VERSION=4.2.11
ENV IRODS_VERSION=4.2.12

RUN curl -sSL https://packages.irods.org/irods-signing-key.asc | apt-key add - && \
echo "deb [arch=amd64] https://packages.irods.org/apt/ $(lsb_release -sc) main" |\
Expand All @@ -28,20 +29,28 @@ RUN curl -sSL https://packages.irods.org/irods-signing-key.asc | apt-key add - &
irods-runtime="${IRODS_VERSION}-1~$(lsb_release -sc)" \
irods-icommands="${IRODS_VERSION}-1~$(lsb_release -sc)"

RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys "E1DD270288B4E6030699E45FA1715D88E1DF1F24" && \
echo "deb https://ppa.launchpadcontent.net/git-core/ppa/ubuntu $(lsb_release -sc) main" |\
tee /etc/apt/sources.list.d/git-core.list && \
apt-get update && \
apt-get install -q -y --no-install-recommends \
git

RUN apt-get update && \
apt-get install -q -y --no-install-recommends \
autoconf \
automake \
build-essential \
check \
cmake \
gdb \
git \
jq \
lcov \
less \
libjansson-dev \
libtool \
pkg-config \
python3-sphinx \
ssh \
valgrind \
unattended-upgrades && \
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
services:
irods-server:
container_name: irods-server
image: "ghcr.io/wtsi-npg/ub-18.04-irods-${IRODS_VERSION:-4.2.11}:${DOCKER_TAG:-latest}"
image: "ghcr.io/wtsi-npg/ub-18.04-irods-${IRODS_VERSION:-4.2.12}:${DOCKER_TAG:-latest}"
restart: always
ports:
- "1247:1247"
Expand Down
2 changes: 1 addition & 1 deletion src/baton.c
Original file line number Diff line number Diff line change
Expand Up @@ -756,7 +756,7 @@ int modify_metadata(rcComm_t *conn, rodsPath_t *rods_path,
named_args.attr_value = attr_value;
named_args.attr_units = attr_units;

modAVUMetadataInp_t anon_args;
modAVUMetadataInp_t anon_args = {0};
map_mod_args(&anon_args, &named_args);

int status = rcModAVUMetadata(conn, &anon_args);
Expand Down
1 change: 0 additions & 1 deletion tests/check_baton.c
Original file line number Diff line number Diff line change
Expand Up @@ -1377,7 +1377,6 @@ START_TEST(test_add_metadata_obj) {
}
END_TEST


// Do we fail to add metadata to a non-existent path?
START_TEST(test_add_metadata_missing_path) {
option_flags flags = 0;
Expand Down

0 comments on commit 32c3e38

Please sign in to comment.