-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
*: run kubernetes tests in travis #1072
Conversation
Do you want to add a note here https://github.com/coreos/dex/blob/master/Documentation/dev-integration-tests.md mentioning that we have added these tests to travis? |
storage/kubernetes/storage.go
Outdated
return nil | ||
} | ||
} | ||
return fmt.Errorf("crd %s not ready", err) |
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.
woops, needs to be "name"
@rithujohn191 ready for review |
scripts/test-k8s.sh
Outdated
printf '.' | ||
sleep 1 | ||
done | ||
echo "ok" |
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.
maybe we could print out a more meaningful message here.
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.
Yeah, changed to "API server ready"
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.
LGTM
*: run kubernetes tests in travis
Just an experiment for now