Skip to content
Snippets Groups Projects

Cannot push to migrated Hawk project (wrong login)

    • Closed Issue created by Antonio Garcia-Dominguez @agarciad

    When I log into Gitlab through gitlab.eclipse.org, I am logged in as @agarciad1 and not as @agarciad. This means that I cannot add my usual SSH key to my account, so I cannot push. When I try to push, I get this:

    remote: 
remote: ========================================================================
remote: 
remote: You are not allowed to push code to this project.
remote: 
remote: ========================================================================
remote: 
fatal: Could not read from remote repository.
    Edited

    Designs

      Child items ...

      2. Activity

      • Antonio Garcia-Dominguez changed title from Cannot push to migrated Hawk project to Cannot push to migrated Hawk project (wrong login)

        changed title from Cannot push to migrated Hawk project to Cannot push to migrated Hawk project (wrong login)

      • Antonio Garcia-Dominguez
        Antonio Garcia-Dominguez @agarciad ·
        Author

        BTW, this is the user who is a member of Hawk:

        https://gitlab.eclipse.org/agarciad

      • Antonio Garcia-Dominguez
        Antonio Garcia-Dominguez @agarciad ·
        Author

        Additionally, every time I log in I have to go through the same "Do you want to authorize this application to use your account?" page. Shouldn't it be enough to authorize it once?

      • Denis Roy
        Denis Roy @droy ·
        Reporter

        I was able to change your login to agarciad. Can you try now?

        /cc @cguindon About the session: please see https://bugs.eclipse.org/bugs/show_bug.cgi?id=506474

      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        @droy Can you try now?

      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        /cc @cguindon About the session: please see https://bugs.eclipse.org/bugs/show_bug.cgi?id=506474

        No progress has been done for this on drupal.org: https://www.drupal.org/node/2460805 but this is not a blocker for Gitlab.

      • Denis Roy
        Denis Roy @droy ·
        Reporter

        @cguindon works, thanks for removing the need to confirm the login!

      • Denis Roy
        Denis Roy @droy ·
        Reporter

        @agarciad is your access OK now?

        • Antonio Garcia-Dominguez
          Antonio Garcia-Dominguez @agarciad ·
          Author

          @droy access is fine, but I'm running into a separate problem (it's a pre-receive hook problem now).

          $ git push -u origin master
Enumerating objects: 45, done.
Counting objects: 100% (45/45), done.
Delta compression using up to 4 threads
Compressing objects: 100% (19/19), done.
Writing objects: 100% (27/27), 2.32 KiB | 339.00 KiB/s, done.
Total 27 (delta 15), reused 0 (delta 0)
remote: /opt/gitlab/embedded/service/gitlab-shell/hooks/pre-receive.d/eca.rb:50:in `gsub': invalid byte sequence in US-ASCII (ArgumentError)
remote: 	from /opt/gitlab/embedded/service/gitlab-shell/hooks/pre-receive.d/eca.rb:50:in `block in <main>'
remote: 	from /opt/gitlab/embedded/service/gitlab-shell/hooks/pre-receive.d/eca.rb:43:in `each'
remote: 	from /opt/gitlab/embedded/service/gitlab-shell/hooks/pre-receive.d/eca.rb:43:in `<main>'
        • Martin Lowe
          Martin Lowe @malowe ·
          Reporter

          If possible and it doesn't include any sensitive information, can you include the full commit message? I have a possible fix and synthetic test case, but having a real-world use case is always best if possible! I want to be sure I address the issue you're hitting, not just the 90% case

        • Please register or sign in to reply
      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        This appears to be an encoding problem.

        I created an issue and we will look at this: https://github.com/EclipseFdn/git-eca-rest-api/issues/18

      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        @droy can you deploy the latest version of the eca.rb script? https://raw.githubusercontent.com/EclipseFdn/git-eca-rest-api/master/src/main/rb/eca.rb

      • Denis Roy
        Denis Roy @droy ·
        Reporter

        New ECA was deployed. @agarciad can you please try now?

      • Antonio Garcia-Dominguez
        Antonio Garcia-Dominguez @agarciad ·
        Author

        I was able to push now (thank you!), but I'm seeing these messages now:

        remote: Commit: 41accf96f7b54f3b96bc0f36c30734aa172f622e		✔
remote: 
remote: 	Reviewing commit: 41accf96f7b54f3b96bc0f36c30734aa172f622e
remote: 	Authored by: Antonio García-Domínguez <agarcdomi@gmail.com>
remote: 	The author is a committer on the project.
remote: 	Reviewing commit: 41accf96f7b54f3b96bc0f36c30734aa172f622e
remote: 	Authored by: Antonio García-Domínguez <agarcdomi@gmail.com>
remote: 	The author is a committer on the project.
remote: Any warnings noted above may indicate compliance issues with committer ECA requirements. More information may be found on https://www.eclipse.org/legal/ECA.php
remote: 
remote: 
remote: Commit: 32975427e48f84f20084aedc8ae207542f906e79		✔
remote: 
remote: 	Reviewing commit: 32975427e48f84f20084aedc8ae207542f906e79
remote: 	Authored by: Antonio García-Domínguez <agarcdomi@gmail.com>
remote: 	The author is a committer on the project.
remote: 	Reviewing commit: 32975427e48f84f20084aedc8ae207542f906e79
remote: 	Authored by: Antonio García-Domínguez <agarcdomi@gmail.com>
remote: 	The author is a committer on the project.
remote: Any warnings noted above may indicate compliance issues with committer ECA requirements. More information may be found on https://www.eclipse.org/legal/ECA.php
remote: 
remote: 
remote: Commit: f1bc5d90755b138a457cf1c579a8b683b33c0314		✔
remote: 
remote: 	Reviewing commit: f1bc5d90755b138a457cf1c579a8b683b33c0314
remote: 	Authored by: Antonio García-Domínguez <agarcdomi@gmail.com>
remote: 	The author is a committer on the project.
remote: 	Reviewing commit: f1bc5d90755b138a457cf1c579a8b683b33c0314
remote: 	Authored by: Antonio García-Domínguez <agarcdomi@gmail.com>
remote: 	The author is a committer on the project.
remote: Any warnings noted above may indicate compliance issues with committer ECA requirements. More information may be found on https://www.eclipse.org/legal/ECA.php
remote: 
remote:

        Do we really need these messages if the commits are fine? I don't really mind them, but they seem somewhat redundant. By the way, for some reason each commit is reviewed twice.

      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        @agarciad I would like to keep those messages since this is a new service and they are useful to understand the nature of the issue.

        We will take a look at why each commit is reviewed twice.

      • Denis Roy
        Denis Roy @droy ·
        Reporter

        Are we good to close this?

      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        @droy We just pushed a fix to the eca.rb script the each for commit being reviewed twice.

        Can you re-deploy it please?

        Edited by Christopher Guindon
      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        I think we can close it once that's done!

      • Denis Roy
        Denis Roy @droy ·
        Reporter

        I've re-deployed eca.rb. @cguindon please confirm.

      • Christopher Guindon
        Christopher Guindon @cguindon ·
        Reporter

        LGTM

      • Christopher Guindon closed

        closed

      • Martin Nonnenmacher mentioned in issue #5543

        mentioned in issue #5543

      Please register or sign in to reply

      Copyright © Eclipse Foundation, Inc. All Rights Reserved.     Privacy Policy | Terms of Use | Copyright Agent