Skip to content
Snippets Groups Projects

Add ability to allow list group namespaces to bypass ECA

Merged Martin Lowe requested to merge malowe/main/eca-script-debug into main
3 unresolved threads
  • Add ability to allow list group namespaces to bypass ECA

This bypass is needed for migrating large existing Git repositories, as the ECA will time out on massive requests.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
22 22 '/eclipse/oniro-core/meta-seco-intel',
23 23 '/eclipse/oniro-core/meta-zephyr'
24 24 ]
25 ALLOW_LIST_GROUPS = ['eclipsefdn/it/webdev/eclipse.org-php-archives/']
  • More of an observation but we define projects with a slash at the start of the string but not groups?

  • Author Maintainer

    It's due to how that check is done, as it uses a different method to check it which is less efficient. I didn't want to touch it since it's working, but this check should be a bit more robust as it doesn't rely on the host URL

  • Please register or sign in to reply
  • Christopher Guindon approved this merge request

    approved this merge request

  • merged

  • Martin Lowe mentioned in commit aec721c2

    mentioned in commit aec721c2

    • Author Maintainer

      @mward in the future, if you're migrating large projects with long histories over, updating the ALLOW_LIST_GROUPS added here should make your job far easier. All you have to do is add the path leading to the group you are migrating, and everything under it should have the ECA check skipped.

    • I'll try to keep it in mind.

    • Please register or sign in to reply
  • Author Maintainer

    @droy sorry for the short turnaround, but there is an ECA script update. This adds the ability to bypass ECA for Gitlab groups designated in this file, with currently our new PHP site archive being the only set group. It should make a couple of our migration activities easier for larger projects as well.

  • Please register or sign in to reply
    Loading