Skip to content

Conversation

@tanderegg
Copy link
Contributor

No description provided.

@tanderegg tanderegg marked this pull request as ready for review October 30, 2025 21:06
@tanderegg tanderegg requested a review from gtempus October 30, 2025 21:08
@tanderegg
Copy link
Contributor Author

@gtempus Could you take a look before merging to Staging? This does a number of things, but nothing too crazy: basically setups Github Actions to actually run TF Plan on PR, and TF Apply on merge, and the actual TF changes involve synchronizing the vars with actually deployed infra, while adding access to the K8s cluster using a new method. Note that with this change, merging will actually apply the TF, you can review the plan output here: https://github.com/resource-watch/api-infrastructure/actions/runs/18955004418/job/54129470211?pr=129.

@tanderegg
Copy link
Contributor Author

(I forgot to add, this was all prep to get things in a better state for updating the K8s cluster version, which is the goal)

Copy link

@gtempus gtempus left a comment

Choose a reason for hiding this comment

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

Let's give it a shot, @tanderegg. Thanks for pointing me to the plan.

Copy link
Member

@solomon-negusse solomon-negusse left a comment

Choose a reason for hiding this comment

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

Looks good to me @tanderegg. The eks API stuff is new to me, so just skimmed thru that for familiarity.

@tanderegg tanderegg merged commit e6ef27f into staging Nov 11, 2025
2 checks passed
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.

4 participants