Skip to content

Update device-plugins to go 1.24.4 and drop 'mf' prefix #16

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.

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

Closed

Conversation

byron-marohn
Copy link
Contributor

No description provided.

@byron-marohn byron-marohn force-pushed the byron_initial_githubactions branch from a8ebf48 to 675ca9f Compare June 20, 2025 03:32
@byron-marohn byron-marohn force-pushed the byron-device-plugin-go-ver branch from f565f2b to f5ede69 Compare June 20, 2025 03:32
@byron-marohn byron-marohn force-pushed the byron_initial_githubactions branch from 675ca9f to 3430568 Compare June 20, 2025 04:05
if [[ $? -ne 0 ]]; then
echo "Error: Failed to push the Docker image"
exit 1
fi
fi

# Update the deployment manifest with the newly built image version
perl -p -i -e "s|image:.*mf-device-plugin:.*$|image: '$DOCKER_REPO/mf-device-plugin:$VER'|g" deploy/manifests/maverikflats-device-plugin.yaml
perl -p -i -e "s|image:.*device-plugin:.*$|image: '$DOCKER_REPO/device-plugin:$VER'|g" deploy/manifests/maverikflats-device-plugin.yaml
Copy link
Contributor

Choose a reason for hiding this comment

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

maverickflats again is internal code name..
Lets use : intel-idv-device-plugin

@@ -1,7 +1,7 @@
devicePlugin:
enabled: true
image:
repository: localhost:5000/mf-device-plugin
repository: localhost:5000/device-plugin
Copy link
Contributor

Choose a reason for hiding this comment

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

lets use intel-idv-device-plugin

smitesh-sutaria and others added 2 commits June 20, 2025 10:04
…tions

Initial github actions for automated build & security scans
Signed-off-by: Byron Marohn <byron.marohn@intel.com>
@byron-marohn byron-marohn force-pushed the byron-device-plugin-go-ver branch from f5ede69 to e10970e Compare June 20, 2025 04:57
@byron-marohn byron-marohn deleted the branch byron_initial_githubactions June 20, 2025 04:57
@byron-marohn
Copy link
Contributor Author

Reopened this as #18, this PR had the wrong target branch. It was supposed to update to main once the parent was merged, but apparently didn't.

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