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

fix(e2e): make package-artifacts to be aware of staging repo #3578

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

tadayosi
Copy link
Member

@tadayosi tadayosi commented Aug 26, 2022

It appears that the builder test is failing just because it's taking longer time to download dependencies from staging repo. Let's see if this simple fix can solve the failures.

While it doesn't still fix the original builder e2e failures, it fixes ./script/package_maven_artifacts.sh to make it aware of staging repo and it seems e2e results have become improved.

Release Note

NONE

@tadayosi
Copy link
Member Author

Hmm, builder / builder-it (Spectrum) still doesn't pass. I'll investigate more...

@tadayosi
Copy link
Member Author

Locally it builds fine after enough warm up, so I still believe it's just a timeout issue. One thing I still don't get is why it's not an issue with Buildah and Kaniko and occurs only with Spectrum.

@tadayosi
Copy link
Member Author

OK, I found a broader issue. ./script/package_maven_artifacts.sh needs to be aware of STAGING_RUNTIME_REPO. Let me fix the script.

@tadayosi tadayosi changed the title fix(e2e): builder test when staging fix(e2e): make package-artifacts to be aware of staging repo Aug 26, 2022
@tadayosi
Copy link
Member Author

Now at least upgrade goes green too. Let me merge this.

@tadayosi tadayosi merged commit 21491d2 into apache:main Aug 26, 2022
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.

None yet

2 participants