Skip to content

Settings feature branch#174

Merged
UserNotFound merged 27 commits into
masterfrom
settings-feature-branch
Apr 30, 2026
Merged

Settings feature branch#174
UserNotFound merged 27 commits into
masterfrom
settings-feature-branch

Conversation

@UserNotFound
Copy link
Copy Markdown
Member

@UserNotFound UserNotFound commented Apr 15, 2026

  • aptible_app gains docker_image, private_registry_username, private_registry_password as top-level attributes
  • aptible_endpoint gains 9 new settings attributes (force_ssl, idle_timeout, maintenance_page_url, etc.) with per-endpoint-type validation
  • Migrates aptible_app create/update to use the newer aptibleapi client
  • Adds convergence testing (plan-after-apply, plan-after-import) across all test cases

Replaces and builds upon the previous PR's, that just had settings/senstive_settings maps:

Note: to test this properly, you will have to set_stack_settings Stack.where(name: 'sbx-settings'), { 'sweetness/nonsensitive/SETTINGS_MIGRATION_PHASE' => '1' }, which I have already done on sbx-settings.

Terraform accectance test
Cloning into 'terraform-provider-aptible'...
remote: Enumerating objects: 1331, done.
remote: Counting objects: 100% (633/633), done.
remote: Compressing objects: 100% (285/285), done.
remote: Total 1331 (delta 523), reused 374 (delta 348), pack-reused 698 (from 3)
Receiving objects: 100% (1331/1331), 823.02 KiB | 5.31 MiB/s, done.
Resolving deltas: 100% (897/897), done.
Switched to a new branch 'settings-feature-branch'
Branch 'settings-feature-branch' set up to track remote branch 'settings-feature-branch' from 'origin'.
Testing branch settings-feature-branch at ec7ef2be1f05201648fee9818ffd3fde99a875a7
...

ok  	github.com/aptible/terraform-provider-aptible/aptible	1111.775s
  passes the terraform acceptance suite

Finished in 19 minutes 55 seconds (files took 1.38 seconds to load)
1 example, 0 failures

UserNotFound and others added 7 commits April 15, 2026 17:11
* Bump go client

* Vhost setting support

* Implement updates

* Add docs

* typo

* Review: scope variables properly

* fmt

* any is interface{}
* Update to CreateContext/ReadContext

* Define and read

* Update resourceAppUpdate

* Minimal payload

* Create

* Return?

* Acceptance tests

* DRY create

* DRY the update code

* Docs
@UserNotFound UserNotFound force-pushed the settings-feature-branch branch from 6d3ab76 to a92e507 Compare April 16, 2026 05:57
@promptless
Copy link
Copy Markdown

promptless Bot commented Apr 16, 2026

Promptless prepared a documentation update related to this change.

Triggered by PR #174 (Settings feature branch)

This PR consolidates the Settings feature work from PRs #160 and #162. The documentation update covers the new app deployment attributes (docker_image, private_registry_username, private_registry_password) and endpoint settings configuration.

Review: Document Terraform provider app and endpoint settings attributes

@UserNotFound UserNotFound force-pushed the settings-feature-branch branch from 4d17901 to 8ea6603 Compare April 16, 2026 17:37
@UserNotFound
Copy link
Copy Markdown
Member Author

@UserNotFound
Copy link
Copy Markdown
Member Author

@UserNotFound UserNotFound marked this pull request as ready for review April 21, 2026 18:26
Copy link
Copy Markdown
Member

@madhuravius madhuravius left a comment

Choose a reason for hiding this comment

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

from a code perspective - i have mainly minor requests/nits. feel free to disagree with them

i need to talk to you about one file in particular aptible/resource_endpoint.go. there's a lot of object => map => object stuff going on and i'm not 100% sure why, is it because of client issues or v1/v2 compat (or something ELSE?)

either way let me know. i'm testing the functionality of it now

Comment thread aptible/resource_app.go Outdated
Comment thread aptible/resource_app.go
Comment thread aptible/resource_app.go Outdated
Comment thread aptible/resource_app.go Outdated
Comment thread aptible/resource_endpoint.go
@madhuravius
Copy link
Copy Markdown
Member

madhuravius commented Apr 22, 2026

The only items remaining are:

  • low impact reduce repetition/exhaustiveness of if/else chain with appending errors
  • preference to add prematurely optimized functions (but you could argue it's clearer to have it broken up this way)

LMK if you feel strongly the other way about the above two and happy to approve (i recommend doing it though just in case). This + integration tests make me feel pretty good about this as that seems exhaustive enough.

The only other potential thing I can think of is folks who might get impacted by state migration/update issues. Ex: existing users on OLDER versions of terraform with settings now moving to NEWER versions may have broken terraform and/or state. I can help test this out and verify.

@UserNotFound
Copy link
Copy Markdown
Member Author

Ready for another look!

madhuravius
madhuravius previously approved these changes Apr 22, 2026
madhuravius
madhuravius previously approved these changes Apr 24, 2026
Copy link
Copy Markdown
Member

@madhuravius madhuravius left a comment

Choose a reason for hiding this comment

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

LGTM - thanks for addressing the feedback

@UserNotFound UserNotFound merged commit fa97460 into master Apr 30, 2026
9 checks passed
@UserNotFound UserNotFound deleted the settings-feature-branch branch April 30, 2026 16:39
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.

2 participants