Skip to content
This repository was archived by the owner on Jan 8, 2024. It is now read-only.

Commit 8cedcae

Browse files
Merge pull request #4103 from hashicorp/backport/issue-3989/lowercase-ids/blatantly-casual-doe
This pull request was automerged via backport-assistant
2 parents 5faef15 + 87a65dd commit 8cedcae

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

.changelog/4063.txt

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
```release-note:bug
2+
cli: Remove automatic uppercasing of ids, so that future runner profiles will match
3+
```

internal/cli/runner_install.go

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -285,14 +285,14 @@ func (c *RunnerInstallCommand) Run(args []string) int {
285285

286286
// Creating a new runner profile for the newly adopted runner
287287
var odrConfig *pb.OnDemandRunnerConfig
288-
s = sg.Add("Creating runner profile and targeting runner %s", strings.ToUpper(id))
288+
s = sg.Add("Creating runner profile and targeting runner %s", id)
289289
if odc, ok := p.(installutil.OnDemandRunnerConfigProvider); ok {
290290
odrConfig = odc.OnDemandRunnerConfig()
291-
odrConfig.Name = odrConfig.Name + "-" + strings.ToUpper(id)
291+
odrConfig.Name = odrConfig.Name + "-" + id
292292
odrConfig.OciUrl = c.runnerProfileOdrImage // Use what we got from flags (or the default)
293293
} else {
294294
odrConfig = &pb.OnDemandRunnerConfig{
295-
Name: platform[0] + "-" + strings.ToUpper(id),
295+
Name: platform[0] + "-" + id,
296296
OciUrl: c.runnerProfileOdrImage,
297297
PluginType: platform[0],
298298
}
@@ -308,9 +308,7 @@ func (c *RunnerInstallCommand) Run(args []string) int {
308308
} else {
309309
odrConfig.TargetRunner = &pb.Ref_Runner{
310310
Target: &pb.Ref_Runner_Id{
311-
Id: &pb.Ref_RunnerId{
312-
Id: strings.ToUpper(id),
313-
},
311+
Id: &pb.Ref_RunnerId{Id: id},
314312
},
315313
}
316314
}

0 commit comments

Comments
 (0)