Skip to content

Commit 8614ed8

Browse files
tphoneyactions-user
authored andcommitted
fix, use ovm.change.uuid in cli and api-server (#2793)
<!-- CURSOR_SUMMARY --> > [!NOTE] > Standardizes log field key for change UUID to `ovm.change.uuid` in API server and CLI change commands. > > - **Logging**: > - Standardize change UUID field to `ovm.change.uuid`. > - API Server (`api-server/server/changesservice.go`): update log fields in start/end change flows and related messages. > - CLI: > - `cli/cmd/changes_get_change.go`: log "found change" with `ovm.change.uuid`. > - `cli/cmd/changes_list_changes.go`: use `ovm.change.uuid` in info logs and error field contexts for related snapshot/bookmark fetches. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 871333da7a92fc082c5c7250ecd166ad5968ed1d. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> GitOrigin-RevId: 5def9c3d125225495bf2d036032da5024171b309
1 parent 4c8c9da commit 8614ed8

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

cmd/changes_get_change.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ fetch:
146146
}
147147
}
148148
log.WithContext(ctx).WithFields(log.Fields{
149-
"change-uuid": changeUuid.String(),
149+
"ovm.change.uuid": changeUuid.String(),
150150
}).Info("found change")
151151

152152
fmt.Println(changeRes.Msg.GetChange())

cmd/changes_list_changes.go

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
4646
for _, change := range response.Msg.GetChanges() {
4747
changeUuid := uuid.UUID(change.GetMetadata().GetUUID())
4848
log.WithContext(ctx).WithFields(log.Fields{
49-
"change-uuid": changeUuid,
49+
"ovm.change.uuid": changeUuid,
5050
"change-created": change.GetMetadata().GetCreatedAt().AsTime(),
5151
"change-status": change.GetMetadata().GetStatus().String(),
5252
"change-name": change.GetProperties().GetTitle(),
@@ -80,7 +80,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
8080
return loggedError{
8181
err: err,
8282
fields: log.Fields{
83-
"change-uuid": changeUuid,
83+
"ovm.change.uuid": changeUuid,
8484
"changing-items-uuid": ciUuid.String(),
8585
},
8686
message: "failed to get ChangingItemsBookmark",
@@ -92,7 +92,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
9292
return loggedError{
9393
err: err,
9494
fields: log.Fields{
95-
"change-uuid": changeUuid,
95+
"ovm.change.uuid": changeUuid,
9696
"changing-items-uuid": ciUuid.String(),
9797
},
9898
message: "Error rendering changing items bookmark",
@@ -119,7 +119,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
119119
return loggedError{
120120
err: err,
121121
fields: log.Fields{
122-
"change-uuid": changeUuid,
122+
"ovm.change.uuid": changeUuid,
123123
"blast-radius-uuid": brUuid.String(),
124124
},
125125
message: "failed to get BlastRadiusSnapshot",
@@ -131,7 +131,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
131131
return loggedError{
132132
err: err,
133133
fields: log.Fields{
134-
"change-uuid": changeUuid,
134+
"ovm.change.uuid": changeUuid,
135135
"blast-radius-uuid": brUuid.String(),
136136
},
137137
message: "Error rendering blast radius snapshot",
@@ -158,7 +158,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
158158
return loggedError{
159159
err: err,
160160
fields: log.Fields{
161-
"change-uuid": changeUuid,
161+
"ovm.change.uuid": changeUuid,
162162
"system-before-uuid": sbsUuid.String(),
163163
},
164164
message: "failed to get SystemBeforeSnapshot",
@@ -170,7 +170,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
170170
return loggedError{
171171
err: err,
172172
fields: log.Fields{
173-
"change-uuid": changeUuid,
173+
"ovm.change.uuid": changeUuid,
174174
"system-before-uuid": sbsUuid.String(),
175175
},
176176
message: "Error rendering system before snapshot",
@@ -197,7 +197,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
197197
return loggedError{
198198
err: err,
199199
fields: log.Fields{
200-
"change-uuid": changeUuid,
200+
"ovm.change.uuid": changeUuid,
201201
"system-after-uuid": sasUuid.String(),
202202
},
203203
message: "failed to get SystemAfterSnapshot",
@@ -209,7 +209,7 @@ func ListChanges(cmd *cobra.Command, args []string) error {
209209
return loggedError{
210210
err: err,
211211
fields: log.Fields{
212-
"change-uuid": changeUuid,
212+
"ovm.change.uuid": changeUuid,
213213
"system-after-uuid": sasUuid.String(),
214214
},
215215
message: "Error rendering system after snapshot",

0 commit comments

Comments
 (0)