Skip to content

Commit bef6c4a

Browse files
committed
repo: add region tracing
Add 'TraceLogger.Region()' invocations to all public methods of the 'RepoProvider' struct. Signed-off-by: Victoria Dye <[email protected]>
1 parent 45749e0 commit bef6c4a

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

internal/core/repo.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,9 @@ func NewRepositoryProvider(logger log.TraceLogger,
3939
}
4040

4141
func (r *repoProvider) CreateRepository(ctx context.Context, route string) (*Repository, error) {
42+
ctx, exitRegion := r.logger.Region(ctx, "repo", "create_repo")
43+
defer exitRegion()
44+
4245
user, err := r.user.CurrentUser()
4346
if err != nil {
4447
return nil, err
@@ -79,6 +82,9 @@ func (r *repoProvider) CreateRepository(ctx context.Context, route string) (*Rep
7982
}
8083

8184
func (r *repoProvider) RemoveRoute(ctx context.Context, route string) error {
85+
ctx, exitRegion := r.logger.Region(ctx, "repo", "remove_route")
86+
defer exitRegion()
87+
8288
repos, err := r.GetRepositories(ctx)
8389
if err != nil {
8490
return fmt.Errorf("failed to parse routes file: %w", err)
@@ -112,6 +118,9 @@ func (r *repoProvider) writeRouteFile(repos map[string]Repository) error {
112118
}
113119

114120
func (r *repoProvider) GetRepositories(ctx context.Context) (map[string]Repository, error) {
121+
ctx, exitRegion := r.logger.Region(ctx, "repo", "get_repos") //lint:ignore SA4006 keep ctx up-to-date
122+
defer exitRegion()
123+
115124
user, err := r.user.CurrentUser()
116125
if err != nil {
117126
return nil, err

0 commit comments

Comments
 (0)