Skip to content

Include Default Worker Pool for Azure Web App targets#603

Merged
benPearce1 merged 3 commits into
mainfrom
benpearce/fd-357-sev-3-low-octopus-cli-deployment-target-view-command-does
May 26, 2026
Merged

Include Default Worker Pool for Azure Web App targets#603
benPearce1 merged 3 commits into
mainfrom
benpearce/fd-357-sev-3-low-octopus-cli-deployment-target-view-command-does

Conversation

@benPearce1
Copy link
Copy Markdown
Contributor

Fixes #600

❯ octopus deployment-target azure-web-app view "az target 5" -s default -f json
Name                 az target 5 (Machines-406)
Health status        Unhealthy
Current status       There was a problem communicating with this machine (last checked: Friday, 15 May 2026 11:24:12 am +10:00)
Account              azure fnm sandbox
Web App              testappforfd-339
Environments         Dev
Roles                pi
Tenants              None
Tenant Tags          None
Default Worker Pool  second
❯ octopus deployment-target view "az target" -s default -f json
{
  "Id": "Machines-401",
  "Name": "az target",
  "HealthStatus": "Unhealthy",
  "StatusSummary": "There was a problem communicating with this machine (last checked: Friday, 15 May 2026 11:24:25 am +10:00)",
  "CommunicationStyle": "AzureWebApp",
  "Environments": [
    "Dev"
  ],
  "Roles": [
    "pi"
  ],
  "Tenants": null,
  "TenantTags": [],
  "EndpointDetails": {
    "Web App": "testappforfd-339"
  },
  "WebUrl": "http://localhost:8066/app#/Spaces-1/infrastructure/machines/Machines-401/settings",
  "DefaultWorkerPool": ""
}
❯ octopus deployment-target view "az target 5" -s default -f json
{
  "Id": "Machines-406",
  "Name": "az target 5",
  "HealthStatus": "Unhealthy",
  "StatusSummary": "There was a problem communicating with this machine (last checked: Friday, 15 May 2026 11:24:12 am +10:00)",
  "CommunicationStyle": "AzureWebApp",
  "Environments": [
    "Dev"
  ],
  "Roles": [
    "pi"
  ],
  "Tenants": null,
  "TenantTags": [],
  "EndpointDetails": {
    "Web App": "testappforfd-339"
  },
  "WebUrl": "http://localhost:8066/app#/Spaces-1/infrastructure/machines/Machines-406/settings",
  "DefaultWorkerPool": "second"
}
❯ octopus deployment-target view "cr" -s default -f json
{
  "Id": "Machines-262",
  "Name": "cr",
  "HealthStatus": "Healthy",
  "StatusSummary": "This target is enabled.",
  "CommunicationStyle": "None",
  "Environments": [
    "Dev"
  ],
  "Roles": [
    "pi"
  ],
  "Tenants": null,
  "TenantTags": [],
  "EndpointDetails": {},
  "WebUrl": "http://localhost:8066/app#/Spaces-1/infrastructure/machines/Machines-262/settings"
}

@benPearce1 benPearce1 requested a review from a team May 15, 2026 01:56
@benPearce1 benPearce1 changed the title Benpearce/fd 357 sev 3 low octopus cli deployment target view command does Include Default Worker Pool for Azure Web App targets May 15, 2026
Copy link
Copy Markdown
Contributor

@mik-ky mik-ky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First pass, one question that I'd like to understand.

Comment thread pkg/cmd/target/view/view.go Outdated
Comment thread pkg/cmd/target/view/view.go Outdated
Comment thread pkg/cmd/target/view/view.go Outdated
…OnWorker interface

cleaned up duplicated code between target list and view commands
@benPearce1 benPearce1 requested a review from mik-ky May 20, 2026 06:37
@benPearce1 benPearce1 merged commit 6502516 into main May 26, 2026
4 checks passed
@benPearce1 benPearce1 deleted the benpearce/fd-357-sev-3-low-octopus-cli-deployment-target-view-command-does branch May 26, 2026 03:24
benPearce1 pushed a commit that referenced this pull request May 26, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.21.3](v2.21.2...v2.21.3)
(2026-05-26)


### Bug Fixes

* Include Default Worker Pool for Azure Web App targets
([#603](#603))
([6502516](6502516)),
closes [#600](#600)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default Worker Pool not shown for deployment targets

2 participants