Skip to content
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

Adding oc, kubectl, version check to common run file #4

Merged
merged 2 commits into from
Sep 16, 2021

Conversation

paigerube14
Copy link
Collaborator

Description

This is adding in common functions that all run.sh files will use
It'll verify the oc and kubectl cli are installed and print off the cluster version to verify the kubeconfig is properly defined

Fixes

#2

@paigerube14 paigerube14 changed the title Adding oc, kubectl, version check to common run file [WIP] Adding oc, kubectl, version check to common run file Sep 14, 2021
@paigerube14
Copy link
Collaborator Author

paigerube14 commented Sep 15, 2021

PTAL @chaitanyaenr when you get a chance

Think I pushed all branches to https://quay.io/repository/prubenda/kraken-hub?tab=tags for testing

@paigerube14 paigerube14 changed the title [WIP] Adding oc, kubectl, version check to common run file Adding oc, kubectl, version check to common run file Sep 15, 2021
@@ -0,0 +1,52 @@
version: '3'
Copy link
Collaborator

Choose a reason for hiding this comment

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

We might want instructions in the readme on how to build all the images, individual image of interest using docker-compose/podman-compose. Thoughts?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yeh definitely think we should add a doc! I'll add

@paigerube14
Copy link
Collaborator Author

Added in docker-compose/podman-compose instructions on how to build your own images

PTAL @chaitanyaenr

Copy link
Collaborator

@chaitanyaenr chaitanyaenr left a comment

Choose a reason for hiding this comment

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

LGTM. Nice addition @paigerube14!

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.

2 participants