Skip to content

Add init container to GFD for handling imex nodes config mount#1070

Merged
cdesiniotis merged 3 commits intomainfrom
gfd-imex-init-container
Oct 28, 2024
Merged

Add init container to GFD for handling imex nodes config mount#1070
cdesiniotis merged 3 commits intomainfrom
gfd-imex-init-container

Conversation

@cdesiniotis
Copy link
Contributor

No description provided.

echo "Removing cached IMEX nodes config"
rm -f /config/${IMEX_NODES_CONFIG_FILE}
fi
if [[ ! -f ${DRIVER_ROOT_CTR_PATH}/${IMEX_NODES_CONFIG_FILE} ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is DRIVER_ROOT_CTR_PATH being set?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few lines above when we source the driver-ready file.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here are the logs from the init container now:

cdesiniotis@cnt-imex-devel:~$ kubectl logs -n gpu-operator gpu-feature-discovery-5nd22 -c gpu-feature-discovery-imex-init
IS_HOST_DRIVER=true
NVIDIA_DRIVER_ROOT=/
DRIVER_ROOT_CTR_PATH=/host
NVIDIA_DEV_ROOT=/
DEV_ROOT_CTR_PATH=/host
Copying IMEX nodes config

@cdesiniotis
Copy link
Contributor Author

@ArangoGutierrez @tariq1890 do you know why the GitHub actions are not being triggered here?

@tariq1890 tariq1890 marked this pull request as ready for review October 25, 2024 19:50
@tariq1890
Copy link
Contributor

tariq1890 commented Oct 25, 2024

@ArangoGutierrez @tariq1890 do you know why the GitHub actions are not being triggered here?

Opened PR #1071 to fix this

@cdesiniotis cdesiniotis force-pushed the gfd-imex-init-container branch 2 times, most recently from 43e1a9e to 401c81d Compare October 25, 2024 20:35
Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis cdesiniotis force-pushed the gfd-imex-init-container branch from 401c81d to d2c828b Compare October 25, 2024 20:59
@cdesiniotis cdesiniotis force-pushed the gfd-imex-init-container branch from 1e42c5f to 2117822 Compare October 25, 2024 21:56
Signed-off-by: Christopher Desiniotis <cdesiniotis@nvidia.com>
@cdesiniotis cdesiniotis force-pushed the gfd-imex-init-container branch from 2117822 to 22941b5 Compare October 25, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants