Skip to content

fix: git persistence settings for password auth and github app#177

Merged
mik-ky merged 2 commits intomainfrom
ck/fix-project-persistence-settings
Mar 5, 2026
Merged

fix: git persistence settings for password auth and github app#177
mik-ky merged 2 commits intomainfrom
ck/fix-project-persistence-settings

Conversation

@mik-ky
Copy link
Copy Markdown
Contributor

@mik-ky mik-ky commented Feb 25, 2026

Fixes issues related to git persistence settings for version controlled projects.

  1. VCS project with a username/password authentication panics
  2. VCS project configured with github app connection is not supported

Go client needs to be updated after OctopusDeploy/go-octopusdeploy#403 is merged in

[sc-136808]

How to test

resource "octopusdeploy_project" "usernamepassword-test" {
  name             = "usernamepassword-test-project-1"
  lifecycle_id     = "Lifecycles-xx"
  project_group_id = "ProjectGroups-xx"

  git_username_password_persistence_settings {
    url            =  "some url"
    username       = "some username"
    password       = "some password" # try updating password (must be a valid token)
    base_path      = ".octopus"
    default_branch = "main"
  }
}


resource "octopusdeploy_project" "git-test" {
  name             = "git-test-project-1"
  lifecycle_id     = "Lifecycles-xx"
  project_group_id = "ProjectGroups-xx"

  git_github_app_persistence_settings {
    base_path = ".octopus"
    default_branch = "main"
    github_connection_id = "GitHubAppConnections-xx"
    protected_branches = []
    url = "some url"
  }
}

@mik-ky mik-ky force-pushed the ck/fix-project-persistence-settings branch 3 times, most recently from 4c877d0 to e53620e Compare February 27, 2026 01:18
@mik-ky mik-ky marked this pull request as ready for review February 27, 2026 02:39
@Rose-Northey
Copy link
Copy Markdown
Contributor

Rose-Northey commented Mar 1, 2026

I got a bit stuck reviewing this in my timebox. A couple of things which would help

  • (small) add a link to the shortcut reference (i found it and added it but its not automated outside deploy)
  • How I would reproduce the before and after state - I had a look at the zendesk ticket but there were holes in the setup and spent most of my timebox trying to work this out
  • Seeing a screenshot of terraform outputs from before and after to help me put the code into context

Thanks heaps

@mik-ky mik-ky force-pushed the ck/fix-project-persistence-settings branch from e53620e to d017476 Compare March 2, 2026 22:27
Copy link
Copy Markdown
Contributor

@bec-callow-oct bec-callow-oct left a comment

Choose a reason for hiding this comment

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

Tested and cannot reproduce the issue

@mik-ky mik-ky merged commit c0213c4 into main Mar 5, 2026
18 checks passed
@mik-ky mik-ky deleted the ck/fix-project-persistence-settings branch March 5, 2026 03:41
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.

3 participants