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

[full-ci] Async Postprocessing #5207

Merged
merged 7 commits into from
Dec 16, 2022
Merged

Conversation

kobergj
Copy link
Collaborator

@kobergj kobergj commented Dec 8, 2022

Brings async postprocessing to master

Needs reva cs3org/reva#3531

Mainly backports from experimental #4100

@update-docs
Copy link

update-docs bot commented Dec 8, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@kobergj kobergj force-pushed the AsyncPostprocessing branch from d9e3e90 to 95a67f7 Compare December 8, 2022 14:10
@ownclouders
Copy link
Contributor

ownclouders commented Dec 8, 2022

💥 Acceptance test wopiValidatorTests-ocis failed. Further test are cancelled...

@kobergj kobergj changed the title Async Postprocessing [full-ci] Async Postprocessing Dec 9, 2022
Signed-off-by: jkoberg <[email protected]>
@kobergj kobergj force-pushed the AsyncPostprocessing branch 3 times, most recently from b39e678 to 58255d4 Compare December 9, 2022 14:07
@wkloucek wkloucek force-pushed the AsyncPostprocessing branch 2 times, most recently from 5e0623f to 7173918 Compare December 12, 2022 08:59
@kobergj kobergj force-pushed the AsyncPostprocessing branch from 7173918 to babc119 Compare December 12, 2022 09:20
@kobergj kobergj force-pushed the AsyncPostprocessing branch from babc119 to f632770 Compare December 14, 2022 09:05
@kobergj kobergj marked this pull request as ready for review December 14, 2022 10:58
Signed-off-by: jkoberg <[email protected]>
@kobergj kobergj force-pushed the AsyncPostprocessing branch from f632770 to c762d9d Compare December 14, 2022 11:25
@sonarcloud
Copy link

sonarcloud bot commented Dec 14, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

1.0% 1.0% Coverage
9.3% 9.3% Duplication

Copy link
Contributor

@C0rby C0rby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good generally but I think it would be nice to add implementations to the version and health commands.

@kobergj
Copy link
Collaborator Author

kobergj commented Dec 16, 2022

Good idea. As on reva I will use a second PR for that so this PR doesn't grow too big.

@kobergj kobergj merged commit 341cdc9 into owncloud:master Dec 16, 2022
@kobergj kobergj deleted the AsyncPostprocessing branch December 16, 2022 10:58
ownclouders pushed a commit that referenced this pull request Dec 16, 2022
Merge: b75e97e c762d9d
Author: kobergj <[email protected]>
Date:   Fri Dec 16 11:58:24 2022 +0100

    Merge pull request #5207 from kobergj/AsyncPostprocessing

    [full-ci] Async Postprocessing
@micbar micbar mentioned this pull request May 3, 2023
89 tasks
@ScharfViktor ScharfViktor mentioned this pull request May 4, 2023
86 tasks
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.

4 participants