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

CircleCI integration tests broken - master-latest-daily charts not found #11853

Closed
howardjohn opened this issue Feb 19, 2019 · 10 comments
Closed
Assignees

Comments

@howardjohn
Copy link
Member

CircleCI integration tests are failing with error:

Error: Looks like "https://storage.googleapis.com/istio-prerelease/daily-build/master-latest-daily/charts" is not a valid chart repository or cannot be reached: Failed to fetch https://storage.googleapis.com/istio-prerelease/daily-build/master-latest-daily/charts/index.yaml : 404 Not Found

Example: https://circleci.com/gh/istio/istio/330161?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link

@ymesika
Copy link
Member

ymesika commented Feb 19, 2019

I guess the easiest path to fixing it is to merge release-1.1 into master.
Otherwise do some cherry-picking (#11805 specifically with all of @sdake helm changes).

@louiscryan
Copy link
Contributor

louiscryan commented Feb 19, 2019 via email

@sdake
Copy link
Member

sdake commented Feb 20, 2019

The release-1.1 charts are here:
helm repo add istio.io "https://gcsweb.istio.io/gcs/istio-prerelease/daily-build/release-1.1-latest-daily/charts/

@sdake
Copy link
Member

sdake commented Feb 20, 2019

As far as I can tell, the master daily charts release process is busted and is not being automatically pushed by the infrastructure. It looks like the bucket has been removed. They were previously at this address:

https://storage.googleapis.com/istio-prerelease/daily-build/master-latest-daily/charts

See:
https://github.com/istio/istio.io/pull/3249/files

To get things unblocked, I'd recommend just using the release-1.1 chart address in #11853 (comment) and hunt down the change that broke the gate.

One of the problems with our process of merging in release-1.1 without a postsubmit gate is software isn't gated, and I suspect not built nor pushed automatically - but I am not certain.

@linsun any ideas?

@pitlv2109
Copy link
Member

Is anyone working on fixing this issue? Individual PR or branch can add a workaround by using a working charts repo, e.g. release-1.1-latest-daily/charts/, but I think an official fix would be ideal. Thanks!

@irisdingbj
Copy link
Member

I believe @louiscryan is working on a fix for the problem

@utka
Copy link
Contributor

utka commented Feb 20, 2019

The reason is we can not build master since 17 Feb, and the master-latest-build was deleted for latest available build 16th. I think it is deleted by script before pushing new version, but because new version does not exist, it fails.
https://storage.googleapis.com/istio-prow/logs/release-daily/216/build-log.txt

@utka
Copy link
Contributor

utka commented Feb 20, 2019

I am triggering daily build manually now

@utka
Copy link
Contributor

utka commented Feb 20, 2019

I manually triggered
gsutil -q cp -r 'gs://istio-prerelease/daily-build/master-20190216-09-16/*' gs://istio-prerelease/daily-build/master-latest-daily/

So master-latest-daily exists now, still going to trigger new daily release

@ozevren
Copy link
Contributor

ozevren commented Feb 22, 2019

Can we close this?

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

No branches or pull requests

8 participants