Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add .gitlab-ci.yml #10

Closed
wants to merge 97 commits into from
Closed

add .gitlab-ci.yml #10

wants to merge 97 commits into from

Conversation

teeverr
Copy link

@teeverr teeverr commented May 31, 2024

Description of your changes

Fixes #

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

dependabot bot and others added 30 commits January 23, 2024 11:29
Bumps [tj-actions/changed-files](https://github.com/tj-actions/changed-files) from 34 to 41.
- [Release notes](https://github.com/tj-actions/changed-files/releases)
- [Changelog](https://github.com/tj-actions/changed-files/blob/main/HISTORY.md)
- [Commits](tj-actions/changed-files@v34...v41)

---
updated-dependencies:
- dependency-name: tj-actions/changed-files
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps google.golang.org/protobuf from 1.31.0 to 1.33.0.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Henrique Oliveira <[email protected]>
Signed-off-by: Maximilian Blatt (external expert on behalf of DB Netz) <[email protected]>
Signed-off-by: Henrique Oliveira <[email protected]>
feat(ec2/securitygroup)!: Make region a required field
fix(efs): Add poll-interval to EFS mount target
…t-ignore-aws-system-tags

fix(s3 bucket): Ignore AWS system tags
…pendabot/go_modules/google.golang.org/protobuf-1.33.0

chore(deps): bump google.golang.org/protobuf from 1.31.0 to 1.33.0
…pendabot/github_actions/dot-github/workflows/tj-actions/changed-files-41

chore(deps): bump tj-actions/changed-files from 34 to 41 in /.github/workflows
…cket-ignore-aws-system-tags

fix(s3 bucket): hotfix for ignoring AWS system tags
Resource tags are used to ensure that only IAM roles with the right
access can modify the resources. This ensures that the security groups
will have the right tags from creation time. Without this change,
IAM policy based on tags may fail to run subsequent steps of security
group creation.

Signed-off-by: justin.miron <[email protected]>
This ensures that when tags change, the EC2 instance reconciles the
update. Previously the ec2 instance controller supported updating tags
via `CreateTags` but did not perform reconciliation when tags changed as
it was not included in the check for if the EC2 instance was up to date.

However, the way `Update` is currently implemented had the implicit
assumption that the EC2 instance must be stopped for all updates, which
is true for all the ModifyInstanceAttribute calls, however, tags can be
applied at runtime. This changes the logic of the controller to (1)
include the instance attributes in the instance observations, (2) only
attempt to modify instance attributes if the observation does not match
the spec.

Signed-off-by: justin.miron <[email protected]>
The EC2 instance controller synchronously makes several
DescribeInstanceAttribut calls to the EC2 API and does not use the
results. This slows down the reconcile time for a single observation.

This removes the API calls for attributes that are returned in
`RunInstances`.

Signed-off-by: justin.miron <[email protected]>
…e-attributes

ec2: include attributes in instance observation, fix reconciling tags
…tion-time-sg

ec2: Create security group with tags in the spec
Signed-off-by: Charel Baum (external expert on behalf of DB InfraGO AG) <[email protected]>
…ce-updateloop-restorefrom

fix(database): ignore restoreFrom in isUpToDate
…controller-api-calls

Reduce ec2 instance controller API calls for an observation
…torageThroughput

Signed-off-by: Charel Baum (external expert on behalf of DB InfraGO AG) <[email protected]>
For records that have MultiValueAnswer and SetIdentifier fields.

Signed-off-by: Max Melentyev <[email protected]>
Otherwise AWS returns ALL routes with name > requested one.

Signed-off-by: Max Melentyev <[email protected]>
…e-iops-storagethroughput-isuptodate-fix

fix(dbinstance): in isUpToDate consider 0 and nil to be equal for iops/storageThroughput
…hanges-detection

Fix changes detection for route53.ResourceRecordSet
…pi-call

Call route53.ListResourceRecordSets with MaxItems = 1
…-with-desired-count

fix: take into consideration the DesiredCount/RunningCount for resource readiness
Signed-off-by: Charel Baum (external expert on behalf of DB InfraGO AG) <[email protected]>
…nce-storagethroughput-fix-dbinstance-iops-storagethroughput

feat(rdsinstance): add storageThroughput
Replace imported types with plane strings.

Signed-off-by: Maximilian Blatt (external expert on behalf of DB Netz) <[email protected]>
@teeverr teeverr closed this Aug 8, 2024
@teeverr teeverr deleted the gitlab-ci branch October 3, 2024 15:05
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.

7 participants