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
AGDIGGER-42 - general clean-up in docker setup #5
Conversation
Review @luigizuccarelli @aliok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wtrocki - Reviewed - great job !!!
<serviceAccount>${oc_serviceaccount_name}</serviceAccount> | ||
</org.csanchez.jenkins.plugins.kubernetes.PodTemplate> | ||
<org.csanchez.jenkins.plugins.kubernetes.PodTemplate> | ||
<name>android</name> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wtrocki - As mentioned this is a hardcoded version to get the POC working. We will look at other ways to do this better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this defines node (android)
in the pipeline, spins up an 'Android" container/slave ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! It's actual label field bellow, but of course entire definition allows to use android label. Idea is to have one android image that would work for both :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mentioned ticket: https://issues.jboss.org/browse/AGDIGGER-41
👍 ship it |
Motivation
Cleanup configuration to include:
Note: image already pushed to aerogear repo.
Verification steps
Try to execute local build steps
Publish image to dockerhub (to check if you have permission to do so)
Setup new PersistentVolume
Change DeploymentConfig to include new image
Expected results
Android node should be available by default.