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

Fixes and enhancments #70

Merged
merged 4 commits into from
Nov 19, 2020
Merged

Fixes and enhancments #70

merged 4 commits into from
Nov 19, 2020

Conversation

k0da
Copy link
Contributor

@k0da k0da commented Oct 23, 2020

  • updates Wrangler to 0.7.2
  • Fixes crednetials for git clone
  • update to go 1.15
  • Watch for configmap and secret and reconcile state if related are changed

@luthermonson
Copy link
Contributor

can you address the build fail @k0da https://drone-pr.rancher.io/rancher/terraform-controller/84/1/2

@k0da
Copy link
Contributor Author

k0da commented Oct 23, 2020

@luthermonson sure, looking at them

@luthermonson luthermonson self-requested a review October 23, 2020 17:50
@k0da k0da force-pushed the wrangler_update branch 3 times, most recently from f2ebc24 to 63b038a Compare October 27, 2020 00:15
Signed-off-by: Dinar Valeev <dinar.valeev@absa.africa>
@k0da k0da force-pushed the wrangler_update branch 3 times, most recently from 2a554ce to fd9eafa Compare October 27, 2020 07:43
@k0da k0da force-pushed the wrangler_update branch 3 times, most recently from 481d1c4 to 4d22df7 Compare November 3, 2020 10:39
* update GO to 1.15
* EnqueueAfter intervalSeconds - periodically watch git for new commits
* Drop usage of deprecated UpdateOnChange from wrangler
* Drop wrangler-api
* Update golangci-lint to 1.18

Signed-off-by: Dinar Valeev <dinar.valeev@absa.africa>
@k0da k0da force-pushed the wrangler_update branch 2 times, most recently from 2f90162 to f48721a Compare November 16, 2020 19:02
k0da added 2 commits November 16, 2020 20:18
Signed-off-by: Dinar Valeev <dinar.valeev@absa.africa>
* Enqueue state if Secret or Configmaps holding TF vars are changed

Signed-off-by: Dinar Valeev <dinar.valeev@absa.africa>
@luthermonson luthermonson merged commit 7fd29d1 into rancher:master Nov 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants