Skip to content
Snippets Groups Projects
  1. Mar 12, 2021
    • Georges Racinet's avatar
      Setting version for release · 79d8f8a8df49
      Georges Racinet authored
      0.10.0
      79d8f8a8df49
    • Georges Racinet's avatar
      RepositoryService.WriteRef: special ref target can be any revision · 28d8fc467b7a
      Georges Racinet authored
      Found out about this while testing with the Rails app. Of course,
      it is natural for symbolic refs, and doesn't sound very logical
      with normal refs, a result of under-specification ultimately due
      to the fact that Gitaly simply uses `git update-ref`.
      
      We keep refusing ref target revisions for keep-arounds because
      we are much surer that the Rails app will only use full SHAs as
      target revisions. Also, it swallows exceptions, avoiding failures
      if `WriteRef` refuses to create the keep-around
      (see `Gitlab::Git::KeepAround`).
      
      We should have a Gitaly comparison test for this, but we already
      have RSpec tests in the Rails app, so this can be in a follow-up.
      28d8fc467b7a
    • Georges Racinet's avatar
      Bumping future version · 5cc50c9c21ca
      Georges Racinet authored
      The work to support special refs is well worth a minor version
      update.
      5cc50c9c21ca
  2. Mar 11, 2021
  3. Mar 09, 2021
  4. Mar 10, 2021
  5. Mar 03, 2021
  6. Mar 10, 2021
  7. Mar 03, 2021
  8. Mar 11, 2021
  9. Mar 09, 2021
  10. Mar 03, 2021
  11. Mar 06, 2021
  12. Mar 04, 2021
    • Georges Racinet's avatar
      DiffService: accepting EMPTY_TREE_OID to mean no commit · b6984e1331b3
      Georges Racinet authored
      This is the convention used by the Rails application, for
      instance in
      
      ```ruby
            def diff_from_parent_request_params(commit, options = {})
              parent_id = commit.parent_ids.first || Gitlab::Git::EMPTY_TREE_ID
      
              diff_between_commits_request_params(parent_id, commit.id, options)
            end
      ```
      
      Without such a special case, HGitaly was returning an error.
      This change gets at least one of the Heptapod functional tests to pass.
      
      Testing shows that Gitaly also accepts the right commit id to be
      EMPTY_TREE_OID to express the absence of commit. This is probably
      less useful, but we're also implementing it for the sake of
      completeness.
      b6984e1331b3
    • Georges Racinet's avatar
      DiffService: better logging of errors from common resolving code · 71d814317a3c
      Georges Racinet authored
      Since `parse_diff_request` is used by several RPCs, it is useful
      for the error logging to include indication of which one was
      called. Using the request class name does the trick.
      71d814317a3c
  13. Mar 05, 2021
  14. Feb 24, 2021
  15. Mar 01, 2021
    • Georges Racinet's avatar
      message.commit_for_tree · 442a6329c053
      Georges Racinet authored
      This is a shortcut, simpler than the other ones in `message`
      but the fully qualified name,
      `ListLastCommitsForTreeResponse.CommitForTree`
      makes hard to respect the 80 columns mark.
      442a6329c053
    • Georges Racinet's avatar
      manifest: new module and classes for manifest mining · f774384f4849
      Georges Racinet authored
      Some repeated patterns of data extraction have been spotted
      in the current ongoing experiments. That's not surprising:
      Gitaly is after all modelled on Git responses.
      f774384f4849
  16. Feb 24, 2021
  17. Feb 27, 2021
Loading