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

Revert "Add CI for autogeneration of api docs on push to main" #312

Merged
merged 1 commit into from
Jun 11, 2021

Conversation

nitishm
Copy link
Contributor

@nitishm nitishm commented Jun 11, 2021

Reverts #310 so that we can get the api.md docs back on the website

@nitishm nitishm merged commit 66485c1 into main Jun 11, 2021
@nitishm nitishm deleted the revert-310-docs-ci branch June 11, 2021 22:27
@mahalrs
Copy link
Contributor

mahalrs commented Jun 12, 2021

@nitishm All these tests are also failing locally. I'm not sure but this might be because of #284.

@nitishm
Copy link
Contributor Author

nitishm commented Jun 12, 2021

Logs state that it fails to pull ambassador chart. Will try to run with debugger over the weekend

@nitishm
Copy link
Contributor Author

nitishm commented Jun 12, 2021

@mahalrs @jonathan-innis Seems like ambassador registry has some issues - I tried the same pull manually and it failed with a 404 as well

~/github/azure/orkestra helm pull https://www.getambassador.io/helm/ambassador-6.6.0.tgz                                                                      
Error: failed to fetch https://www.getambassador.io/helm/ambassador-6.6.0.tgz : 404 Not Found

😕

Tried a commit prior to #284 and that fails with the same error as well.

@mahalrs
Copy link
Contributor

mahalrs commented Jun 12, 2021

@mahalrs @jonathan-innis Seems like ambassador registry has some issues - I tried the same pull manually and it failed with a 404 as well

~/github/azure/orkestra helm pull https://www.getambassador.io/helm/ambassador-6.6.0.tgz                                                                      
Error: failed to fetch https://www.getambassador.io/helm/ambassador-6.6.0.tgz : 404 Not Found

😕

Tried a commit prior to #284 and that fails with the same error as well.

Maybe this is the reason there were random e2e failures in the last few days. Should we consider serving all charts from https://nitishm.github.io/charts

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