-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Add test for updated CA bundle #14373
Conversation
/backport to stable15 |
/backport to stable14 |
https://curl.haxx.se/docs/caextract.html I guess they are not to happy if you fetch the file multiple times a day. There are some recommendations at "Automated downloads from here". |
Problem is that the mtime is "now" when doing a fresh git clone. Let me have a look into this. |
Any other ideas? :/ |
|
That one makes most sense 👍 |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense 👍
https://drone.nextcloud.com/nextcloud/server/16761/86 (branch name: ci/noid/add-bundle-test) ✔️ |
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Bundle properly skipped - let me rebase for squash all the commits in here. |
a892379
to
2bbef58
Compare
@kesselb Ready for final review 👍 |
https://drone.nextcloud.com/nextcloud/server/16769/86 Looks like $DRONE_BRANCH is the target branch. We can either check against DRONE_REFSPEC for version/noid/ or check if DRONE_BRANCH is stable(\d){2}. |
Not an option because this is true for every backport ... 👿 |
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 16776: failureTESTS=acceptance, TESTS-ACCEPTANCE=app-files
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=app-files-sharing-link
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=app-files-tags
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=login
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=users
Show full log
|
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 16778: failureTESTS=acceptance, TESTS-ACCEPTANCE=app-comments
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=app-files
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=app-files-sharing
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=login
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=users
Show full log
|
https://drone.nextcloud.com/nextcloud/server/16778/86 looks good. checker executed for version/noid/xxx |
🤖 beep boop beep 🤖 Here are the logs for the failed build: Status of 16777: failureTESTS=acceptance, TESTS-ACCEPTANCE=app-files
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=login
Show full log
TESTS=acceptance, TESTS-ACCEPTANCE=users
Show full log
|
Then let's merge it. 👍 From my side for your code. |
backport to stable15 in #14546 |
backport to stable14 in #14547 |
For #14369