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.

Sign up for GitHub

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

Jump to bottom

Kubernetes Invoker tests deploy to OpenShift cluster if logged in #12915

Closed
gsmet opened this issue Oct 24, 2020 · 7 comments · Fixed by #12932
Closed

Kubernetes Invoker tests deploy to OpenShift cluster if logged in #12915

gsmet opened this issue Oct 24, 2020 · 7 comments · Fixed by #12932
Labels
area/kubernetes kind/bug Something isn't working
Milestone
1.10.0.CR1

Comments

@gsmet
Copy link
Member

gsmet commented Oct 24, 2020

The Kubernetes Invoker tests deploy to an OpenShift cluster if you're logged in to one and fail:

[ERROR] The following builds failed:
[ERROR] *  kubernetes-docker-build-and-deploy/pom.xml

We need to fix that and log out from the cluster before running these tests.

/cc @geoand @iocanel
@gsmet gsmet added the kind/bug Something isn't working label Oct 24, 2020
@quarkusbot
Copy link

/cc @geoand

@geoand
Copy link
Contributor

geoand commented Oct 24, 2020

Makes sense

@gsmet
Copy link
Member Author

gsmet commented Oct 24, 2020

BTW, maybe we just need to override some environment variables in the build.

@geoand
Copy link
Contributor

geoand commented Oct 24, 2020

Could be. I don't really remember what these tests do, so I'd have to check it out

@gsmet
Copy link
Member Author

gsmet commented Oct 24, 2020

They are trying to deploy to a cluster and they are supposed to fail because you're not connected.

@geoand
Copy link
Contributor

geoand commented Oct 24, 2020

Right. I just don't remember what their original purpose is.

geoand added a commit that referenced this issue Oct 26, 2020
@geoand
Merge pull request #12932 from geoand/#12915
472dbd3 
Ensure that k8s integration tests don't use locally configured cluster
@geoand geoand mentioned this issue Oct 27, 2020
Merged
@geoand
Copy link
Contributor

geoand commented Oct 27, 2020

Fixed by #12932

@geoand geoand closed this as completed Oct 27, 2020
@geoand geoand added this to the 1.10 - master milestone Oct 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/kubernetes kind/bug Something isn't working
Projects
None yet
Milestone
1.10.0.CR1
Development

Successfully merging a pull request may close this issue.

Ensure that k8s integration tests don't use locally configured cluster geoand/quarkus
3 participants
@gsmet @geoand @quarkusbot