diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2025-11-25 14:46:21 +0100 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2025-11-26 11:31:15 +0000 |
| commit | 951f29f3fa85dd9109f4856608a4a18949f67776 (patch) | |
| tree | 09f5e6d8f84e0f2a9a5d2cd84ba029811a423a6a /syz-cluster | |
| parent | 7fd3eae17beda566d0838b6b6fedf5fedc91c2ee (diff) | |
syz-cluster: don't use custom git tree tags
Since remote branches are already mapped to "tree/name" local branches,
there's no need for the special "tree-HEAD" tag anymore. Simplify the
code.
Diffstat (limited to 'syz-cluster')
| -rw-r--r-- | syz-cluster/kernel-disk/fetch-kernels-template.yaml | 1 | ||||
| -rw-r--r-- | syz-cluster/pkg/triage/git.go | 2 | ||||
| -rw-r--r-- | syz-cluster/pkg/triage/git_test.go | 8 |
3 files changed, 5 insertions, 6 deletions
diff --git a/syz-cluster/kernel-disk/fetch-kernels-template.yaml b/syz-cluster/kernel-disk/fetch-kernels-template.yaml index b99be23dd..0f49e432f 100644 --- a/syz-cluster/kernel-disk/fetch-kernels-template.yaml +++ b/syz-cluster/kernel-disk/fetch-kernels-template.yaml @@ -72,4 +72,3 @@ spec: git remote add ${NAME} ${REPO} git fetch ${NAME} ${BRANCH} --tags "+refs/heads/*:refs/heads/${NAME}/*" "+refs/tags/*:refs/tags/${NAME}/*" git log -1 --format="%h %an %ad %s" ${NAME}/${BRANCH} - git tag -f ${NAME}-head ${NAME}/${BRANCH} diff --git a/syz-cluster/pkg/triage/git.go b/syz-cluster/pkg/triage/git.go index 7eab776c9..390bd11b1 100644 --- a/syz-cluster/pkg/triage/git.go +++ b/syz-cluster/pkg/triage/git.go @@ -30,7 +30,7 @@ func NewGitTreeOps(dir string, sandbox bool) (*GitTreeOps, error) { func (ops *GitTreeOps) HeadCommit(tree *api.Tree) (*vcs.Commit, error) { // See kernel-disk/cron.yaml. - return ops.Git.Commit(tree.Name + "-head") + return ops.Git.Commit(tree.Name + "/" + tree.Branch) } func (ops *GitTreeOps) Commit(treeName, commitOrBranch string) (*vcs.Commit, error) { diff --git a/syz-cluster/pkg/triage/git_test.go b/syz-cluster/pkg/triage/git_test.go index c9e6c5472..31666b7df 100644 --- a/syz-cluster/pkg/triage/git_test.go +++ b/syz-cluster/pkg/triage/git_test.go @@ -20,18 +20,18 @@ func TestGitTreeOpsHead(t *testing.T) { osutil.WriteFile(filepath.Join(baseDir, "file.txt"), []byte("Some content")) repo.Git("add", "file.txt") head1 := repo.CommitChange("first head") - repo.SetTag("mainline-head") + repo.SetTag("mainline/main") osutil.WriteFile(filepath.Join(baseDir, "file.txt"), []byte("Another content")) repo.Git("add", "file.txt") head2 := repo.CommitChange("second head") - repo.SetTag("second-head") + repo.SetTag("second/main") // Verify that the right commits are queried. ops, err := NewGitTreeOps(baseDir, false) assert.NoError(t, err) - commit, err := ops.HeadCommit(&api.Tree{Name: "mainline"}) + commit, err := ops.HeadCommit(&api.Tree{Name: "mainline", Branch: "main"}) assert.NoError(t, err) assert.Equal(t, head1.Hash, commit.Hash) - commit, err = ops.HeadCommit(&api.Tree{Name: "second"}) + commit, err = ops.HeadCommit(&api.Tree{Name: "second", Branch: "main"}) assert.NoError(t, err) assert.Equal(t, head2.Hash, commit.Hash) } |
