Skip to content

[tes-only] change restore cache for e2e#7763

Closed
ScharfViktor wants to merge 1 commit intomasterfrom
changeRestoreCache
Closed

[tes-only] change restore cache for e2e#7763
ScharfViktor wants to merge 1 commit intomasterfrom
changeRestoreCache

Conversation

@ScharfViktor
Copy link
Copy Markdown
Contributor

this PR #7748 included a link to the password for the e2e test.
but now the e2e test doesn't work because we don't have the required password window, as when ocis uses "OCIS_SHARING_PUBLIC_SHARE_MUST_HAVE_PASSWORD": False

example:
https://drone.owncloud.com/owncloud/ocis/28961/40/12
https://drone.owncloud.com/owncloud/ocis/28964/60/11
https://drone.owncloud.com/owncloud/ocis/28976/40/12

it's really strange, It seems that the server is somehow misconfigured in CI or it is some kind of cache problem.
@saw-jan do you have ideas why it happens?

@update-docs
Copy link
Copy Markdown

update-docs Bot commented Nov 20, 2023

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.

@sonarqubecloud
Copy link
Copy Markdown

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 0 Code Smells

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

@ScharfViktor
Copy link
Copy Markdown
Contributor Author

Or just need to do a rebase

I rebased for example #7535 which was failing in CI.
now e2e in the CI are green https://drone.owncloud.com/owncloud/ocis/28978/60/11

@saw-jan
Copy link
Copy Markdown
Member

saw-jan commented Nov 21, 2023

it's really strange, It seems that the server is somehow misconfigured in CI or it is some kind of cache problem.

ocis cache is built per ci build so probably due to out-dated commits history in PRs

@ScharfViktor ScharfViktor deleted the changeRestoreCache branch November 23, 2023 06:35
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