Skip to content
This repository has been archived by the owner on Jul 23, 2020. It is now read-only.

Auto update of planner version to fabric8-ui is broken #2825

Closed
nimishamukherjee opened this issue Mar 29, 2018 · 13 comments
Closed

Auto update of planner version to fabric8-ui is broken #2825

nimishamukherjee opened this issue Mar 29, 2018 · 13 comments

Comments

@nimishamukherjee
Copy link
Collaborator

After #2749 was fixed, we discovered another issue.

Auto update of planner version to fabric8-ui is broken.

Error on fabric8-npm-dependencies:
https://jenkins.cd.test.fabric8.io/blue/organizations/jenkins/fabric8-ui%2Ffabric8-npm-dependencies/detail/master/178/pipeline

@pranavgore09
Copy link
Collaborator

pranavgore09 commented Mar 29, 2018

@jaseemabid I know you are aware of it but just mentioning here for others that changes made for planner's workaround are needed in here as well

@jaseemabid
Copy link
Contributor

@pranavgore09 Noted. I'll patch npmUpdateOrganisationDependencies.groovy as well.

@joshuawilson
Copy link
Member

Can you fix all the build scripts for each of the repos in https://github.com/fabric8-ui

@rupalibehera
Copy link
Collaborator

Here is the PR fabric8io/fabric8-pipeline-library#396 from @jaseemabid , need to test it and merge it.

@joshuawilson
Copy link
Member

I thought this was NOT going to get merged due to the risk of breaking upstream external projects.

@joshuawilson
Copy link
Member

This has started working again, magically.
This also should not be a P0 as the user never sees this.

@piyush-garg
Copy link
Collaborator

piyush-garg commented Apr 24, 2018

The release of Fabric8 Kubernetes Model is also facing the issue. Reopening it.

@joshuawilson
Copy link
Member

I didn't know we had Fabric8 Kubernetes Model auto releasing to fabric8-ui.

@piyush-garg
Copy link
Collaborator

Here is the link to the job I just started after your comment @joshuawilson https://jenkins.cd.test.fabric8.io/job/fabric8io/job/kubernetes-model/job/master/20/console

@rupalibehera rupalibehera added the priority/P1 Critical label Apr 25, 2018
@pradeepto
Copy link

@joshuawilson This is p0 because it blocks releases of few bits - including the jenkins-openshift-sync plugin which is required by Idler.

@joshuawilson
Copy link
Member

@pradeepto it looks like you are raising an issue not related to the title. The cause and fix may be related. But does this block Planner (or anything else) from pushing to fabric8-ui?

@jaseemabid
Copy link
Contributor

jaseemabid commented Apr 30, 2018

I probably fixed this as part of #3324. Can we reproduce this again?

@piyush1594 We have fixed k8s-model, right?

@piyush-garg
Copy link
Collaborator

piyush-garg commented Apr 30, 2018

@jaseemabid We checked for kubernetes-model and it worked after the fix

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants