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

GitHub Workflow debugger #296

Draft
wants to merge 780 commits into
base: master
Choose a base branch
from
Draft

GitHub Workflow debugger #296

wants to merge 780 commits into from

Conversation

MahdiBaghbani
Copy link
Member

This is only for testing purposes.
Do not merge!

@MahdiBaghbani MahdiBaghbani marked this pull request as draft April 25, 2023 19:54
@MahdiBaghbani
Copy link
Member Author

MahdiBaghbani commented Apr 25, 2023

@michielbdejong @mrvahedi68 can someone please explain to me why we have this behavior? totally random!
Performance testing decides to fail or pass based on a coin flip.

also found it here for more reference, apparently it has nothing to do with our branch.

same branch, same tests, different results: nextcloud#37914

@mrvahedi68
Copy link

That's so bad. I waste a lot of time reproducing this error and in this way faced with other errors and by fixing them don't get the same error as GitHub. We have a ghost in the GitHub I think.

@mrvahedi68
Copy link

I don't have access to rerun this check again but I know some members have and can rerun it. If it was random maybe pass with retry.

skjnldsv and others added 24 commits May 24, 2023 09:02
…error-logging

Avoid db connections when logging db connection errors
Signed-off-by: Varun Patil <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
don't always check if we need to setup the object store root
…nonce

webpack: add nonce to unsupported-browser-redirect chunk
Signed-off-by: jld3103 <[email protected]>
…er-action

fix(files): make open folder a default action
…ystem-address-book-enumeration

fix(carddav): Check enumeration settings for all SAB methods
This reverts commit cb852ef

Signed-off-by: julia.kirschenheuter <[email protected]>
…other-tel

fix(carddav): Make system contact phone number RFC compliant
…guest-users

fix(carddav): Don't show system address book cards to guests
Signed-off-by: Simon L <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
…ions-for-pbkdf2

Increase from 100000 to 600000 iterations for hash_pbkdf2
Signed-off-by: Simon L <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
…rmenu-visibility

adjust active element visibility in the usermenu
szaimen and others added 29 commits June 12, 2023 13:53
…askey-get

fix(caching): Avoid checking existence before fetching
…drop-emptycontentsecuritypolicy-allowinlinescript

chore(appframework)!: Drop \OCP\AppFramework\Http\EmptyContentSecurityPolicy::allowInlineScript
…actions

Add a Dependabot configuration to autoupdate GitHub action versions
Bumps [skjnldsv/read-package-engines-version-actions](https://github.com/skjnldsv/read-package-engines-version-actions) from 1.1 to 2.2.
- [Release notes](https://github.com/skjnldsv/read-package-engines-version-actions/releases)
- [Commits](skjnldsv/read-package-engines-version-actions@v1.1...8205673)

---
updated-dependencies:
- dependency-name: skjnldsv/read-package-engines-version-actions
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [peter-evans/create-or-update-comment](https://github.com/peter-evans/create-or-update-comment) from 2 to 3.
- [Release notes](https://github.com/peter-evans/create-or-update-comment/releases)
- [Commits](peter-evans/create-or-update-comment@v2...c6c9a1a)

---
updated-dependencies:
- dependency-name: peter-evans/create-or-update-comment
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 2 to 3.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](actions/setup-node@v2...64ed1c7)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 2 to 3.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v2...0b7f8ab)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
…ctions/skjnldsv/read-package-engines-version-actions-2.2

chore(deps): Bump skjnldsv/read-package-engines-version-actions from 1.1 to 2.2
…ctions/peter-evans/create-or-update-comment-3
Add the config variable for curl calls ("remote_curl_timeout"). E.g., needed for nextcloud federation.

Signed-off-by: Dominik Fuchß <[email protected]>
Signed-off-by: Dominik Fuchß <[email protected]>
Signed-off-by: Dominik Fuchß <[email protected]>
…imeout

Add config variable for curl timeout
…address-book-nullable-groupmanager

fix(carddav): Make SystemAddressBook::__construct $groupManager argument nullable
Enable version features again with S3 versioning
Signed-off-by: MohammadReza vahedi <[email protected]>
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Psalm found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

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.