Skip to content

[Major] Code Freeze PR #21

Merged
merged 66 commits into from
Apr 22, 2025
Merged

[Major] Code Freeze PR #21

merged 66 commits into from
Apr 22, 2025

Conversation

morga471
Copy link
Contributor

No description provided.

morga471 and others added 5 commits April 1, 2025 13:22
* Add module release process (#19)

* add startup taint

* add namespace and startup taint to template

* templatlize

* fix vals

* fewer things

* add a few missing values

* create ns

* update values

* add nodeSelector and tolerations

* remove digest

* node group

* fmt

* syntax and startupTaints

* fix syntax

* update values

* remove ng selector

* lower cpu requests

* use digest again

* simplify repository

* fix repository

* escape the .

* fix subnets_name

* subnet selector

* use the outputs from karpenter module

* create instance profile true

* cleanup

* without cluster tag

* pass in vpc

* enable debug and disable serviceMonitor

* fix merge issues

* update resources and set eksControlPlane true

* try enable_pod_identity

* update updated upstream

* more options for namings

* shorter

* validate role name length

* validate event rules name length

* shorter

* shorter still

* fix prefix

* name the queue

* 66 - 3 = 63, limit 64

* longest event name is InstanceStateChange

* another

* maybe

* name

* name_prefix

* or_not

* add module release process

* update module source

* update release actions

* Comment out unused values

* Comment out vpc_id variable definition

* Disable IRSA

---------

Co-authored-by: Matthew Creal Morgan <matthew.c.morgan@census.gov>
@morga471 morga471 self-assigned this Apr 16, 2025
@morga471
Copy link
Contributor Author

morga471 commented Apr 16, 2025

{"level":"ERROR","time":"2025-04-16T21:20:40.420Z","logger":"controller","caller":"operator/operator.go:126","message":"ec2 api connectivity check failed","commit":"ad71530","error":"operation error EC2: DescribeInstanceTypes, get identity: get credentials: failed to refresh cached credentials, failed to retrieve credentials, operation error STS: AssumeRoleWithWebIdentity, https response error StatusCode: 403, RequestID: 937cdb63-8d89-4ac6-bd98-8b772575e4d2, api error AccessDenied: Not authorized to perform sts:AssumeRoleWithWebIdentity"}

@morga471 morga471 changed the title Module Exclusions, Module Release Stuff, eECR sourced images, Naming, Cleanup, and Misc [Major] Code Freeze PR Apr 18, 2025
@morga471 morga471 marked this pull request as ready for review April 21, 2025 17:54
Copy link
Contributor

@nangu001 nangu001 left a comment

Choose a reason for hiding this comment

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

LGTM

@morga471 morga471 merged commit 3cb4bb5 into main Apr 22, 2025
1 check passed
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants