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.

Sign up for GitHub

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

Jump to bottom

[tests-only] Demo ci with user keys encryption #266

Closed
wants to merge 1 commit into from
Closed

[tests-only] Demo ci with user keys encryption #266

wants to merge 1 commit into from

Conversation

phil-davis
Copy link
Contributor

No description provided.

@phil-davis phil-davis self-assigned this Apr 30, 2021
@phil-davis phil-davis mentioned this pull request May 2, 2021
Merged
@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from d7d85a4 to 4576e29 Compare May 2, 2021 14:51
@phil-davis
Copy link
Contributor Author

The longest pipeline was 96 minutes - similar to the 90 minutes for master-key encryption. That is reasonable.

@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch 2 times, most recently from d41bbc0 to fbac470 Compare May 3, 2021 09:50
@phil-davis
Copy link
Contributor Author

The latest CI took 91 minutes. That is reasonable and similar to the CI with master-key encryption.

This is an example PR for test demonstration. I will leave it open for now, so that we can easily rebase and rerun it.

@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from fbac470 to 51bd801 Compare May 8, 2021 13:49
@phil-davis
Copy link
Contributor Author

Note: I will run this again tomorrow after owncloud/core#38718 has been merged. That will verify with core master.

@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from 51bd801 to b7cd1e9 Compare May 11, 2021 07:00
@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from b7cd1e9 to a1a500e Compare May 19, 2021 03:30
@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from a1a500e to b9dc521 Compare May 31, 2021 03:21
@phil-davis
Copy link
Contributor Author

This passed after a rebase today - good.
Elapsed time was 110 minutes. That is OK.

@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from b9dc521 to 4673c9e Compare June 17, 2021 03:58
@phil-davis
Copy link
Contributor Author

I rebased this just now. core daily-master-qa from last night is the code for 10.8.0beta1 so it will be good to verify that all is well with user-keys encryption.

@phil-davis
Copy link
Contributor Author

Passes - 🚀

@phil-davis phil-davis force-pushed the demo-ci-with-user-keys-encryption branch from 4673c9e to b66fb64 Compare July 5, 2021 04:04
@phil-davis
Copy link
Contributor Author

Rebased again to run with latest core daily-master-qa, which should be similar to the next 10.8.0 tarball.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 5, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis
Copy link
Contributor Author

Passed - good.

@phil-davis phil-davis mentioned this pull request Nov 5, 2021
Closed
@phil-davis
Copy link
Contributor Author

See #311 for the latest run of this in Nov 2021

@phil-davis phil-davis closed this Nov 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@phil-davis phil-davis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@phil-davis