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

fix: use "ams" quota type in e2e tests #1240

Closed
wants to merge 1,039 commits into from
Closed

Conversation

parametalol
Copy link
Contributor

@parametalol parametalol commented Sep 5, 2023

As per the quota.md:

There are currently two quota backends -
Quota Management with Account Management Service (AMS)
and quota management list (default). The CLI flag --quota-type={ams,quota-management-list} configures the
quota backend at deploy time of fleet manager. Production instances of fleet manager are configured with
--quota-type=ams to verify quota against customer's SKU entitlements.

vladbologa and others added 30 commits March 22, 2023 14:49
* Remove targeted cluster placement strategy.

* Fix the FirstReadyPlacementStrategy to be a pointer
* add migration for default version table

* implemented central default version api endpoints

* allow POST method for full admins

* set current version as default version for migration

* ROX-12188: api target central version (#907)
…gic (#904)

* Remove skip scheduling.

* Add migration to delete the SkipScheduling column.

* Remove final references to skip scheduling. Replace with use of schedulable.

* Remove SkipScheduling, add Schedulable to cluster API and table columns

* Tests

* Fixed tests for placement. Removed supported instance type from cluster search API.

* Fix migration

* Fix migration ID

* Remove unnecessary columns in migrations, per @machi1990's review comments.

* Fix typo

* Update moq file

* Fix unit test

* Fixes

* Remove gorm default value from Schedulable. No columns use default values and, while 'false' may be surprising, the default value should never be used (aside from when this change is initially rolled out until the first reconciliation of the data plane config in saas.yaml is merged).

* Update Schedulable field in database if the field's value changes in the dataplane config

* Use values map to convince Gorm to update

* Update moq

---------

Co-authored-by: Vlad Bologa <[email protected]>
Co-authored-by: Manyanda Chitimbo  <[email protected]>
* Pass the region from the parameter store to the fleetshard sync

* Update terraform_cluster.sh

Use the parameter under the name CLUSTER_REGION, as it is stored with the cluster's parameters, not a "region" parameter group for the fleetshard sync
Bumps [github.com/onsi/ginkgo/v2](https://github.com/onsi/ginkgo) from 2.9.1 to 2.9.2.
- [Release notes](https://github.com/onsi/ginkgo/releases)
- [Changelog](https://github.com/onsi/ginkgo/blob/master/CHANGELOG.md)
- [Commits](onsi/ginkgo@v2.9.1...v2.9.2)

---
updated-dependencies:
- dependency-name: github.com/onsi/ginkgo/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/golang/glog](https://github.com/golang/glog) from 1.1.0 to 1.1.1.
- [Release notes](https://github.com/golang/glog/releases)
- [Commits](golang/glog@v1.1.0...v1.1.1)

---
updated-dependencies:
- dependency-name: github.com/golang/glog
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [sigs.k8s.io/controller-runtime](https://github.com/kubernetes-sigs/controller-runtime) from 0.14.5 to 0.14.6.
- [Release notes](https://github.com/kubernetes-sigs/controller-runtime/releases)
- [Changelog](https://github.com/kubernetes-sigs/controller-runtime/blob/main/RELEASE.md)
- [Commits](kubernetes-sigs/controller-runtime@v0.14.5...v0.14.6)

---
updated-dependencies:
- dependency-name: sigs.k8s.io/controller-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/onsi/gomega](https://github.com/onsi/gomega) from 1.27.3 to 1.27.6.
- [Release notes](https://github.com/onsi/gomega/releases)
- [Changelog](https://github.com/onsi/gomega/blob/master/CHANGELOG.md)
- [Commits](onsi/gomega@v1.27.3...v1.27.6)

---
updated-dependencies:
- dependency-name: github.com/onsi/gomega
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* import rpm gpg key for CentOS

* Update .openshift-ci/e2e-runtime/Dockerfile

Co-authored-by: Yury Kovalev <[email protected]>

---------

Co-authored-by: Yury Kovalev <[email protected]>
* Add ACS Operator installation

* Drop test chart

* Tune operator chart

* Adopt upgrade tests

* Review comments

* Review comments

* drop configmap

* Add TODOs
* use HEAD as tag for fleetshard-sync image

* use rev-parse
* ROX-16317: Extract a reusable data plane deployment workflow.

Also add github action files linter while at it.

* Fix path, hopefully.

* Fix issues found by linter!

* Do not complain on environments with a single cluster

* pass secrets from environment definition

* replace lint action with precommit

* Add actionlint pre-commit hook (#927)

---------

Co-authored-by: Yury Kovalev <[email protected]>
chore: upgraded app-services-sdk-go to Core SDK
…926)

Bumps [github.com/openshift-online/ocm-sdk-go](https://github.com/openshift-online/ocm-sdk-go) from 0.1.325 to 0.1.331.
- [Release notes](https://github.com/openshift-online/ocm-sdk-go/releases)
- [Changelog](https://github.com/openshift-online/ocm-sdk-go/blob/main/CHANGES.md)
- [Commits](openshift-online/ocm-sdk-go@v0.1.325...v0.1.331)

---
updated-dependencies:
- dependency-name: github.com/openshift-online/ocm-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* ROX-13747: Run deploy workflow in dry-run mode as a PR check.

* inherit secrets

* Add support for dev env.

* correct dev cluster name

* Run "helm lint" rather than "helm upgrade --dry-run" in dev as there is no cluster.

* same helm treatment for probe

* missing import
…incompatible (#936)

Bump github.com/docker/docker

Bumps [github.com/docker/docker](https://github.com/docker/docker) from 20.10.23+incompatible to 20.10.24+incompatible.
- [Release notes](https://github.com/docker/docker/releases)
- [Commits](moby/moby@v20.10.23...v20.10.24)

---
updated-dependencies:
- dependency-name: github.com/docker/docker
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/getsentry/sentry-go](https://github.com/getsentry/sentry-go) from 0.18.0 to 0.20.0.
- [Release notes](https://github.com/getsentry/sentry-go/releases)
- [Changelog](https://github.com/getsentry/sentry-go/blob/master/CHANGELOG.md)
- [Commits](getsentry/sentry-go@v0.18.0...v0.20.0)

---
updated-dependencies:
- dependency-name: github.com/getsentry/sentry-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
johannes94 and others added 5 commits August 29, 2023 10:24
* add collumn and API model to store secrets
* add logic to create cipher based on config
* report encrypted secrets
* only report secrets if not stored yet
* set secrete encryption config in dp terraform helm chart
* encode ciphertext with base64 to allow proper JSON communication
Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.44.300 to 1.44.332.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](aws/aws-sdk-go@v1.44.300...v1.44.332)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@parametalol
Copy link
Contributor Author

parametalol commented Sep 5, 2023

Current dependencies on/for this PR:

@parametalol parametalol temporarily deployed to development September 5, 2023 16:14 — with GitHub Actions Inactive
@parametalol parametalol temporarily deployed to development September 5, 2023 16:14 — with GitHub Actions Inactive
@parametalol parametalol temporarily deployed to development September 5, 2023 16:14 — with GitHub Actions Inactive
@parametalol parametalol changed the title 0x656b694d/e2e-quota-type fix: use "ams" quota type in e2e tests Sep 5, 2023
@stehessel
Copy link
Contributor

Is this test being executed currently anywhere? Sounds like it should be failing?

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 5, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 0x656b694d, stehessel
Once this PR has been reviewed and has the lgtm label, please assign simonbaeumer for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link
Member

@kurlov kurlov 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 drop these command instead 🔥

@openshift-ci openshift-ci bot removed the lgtm label Sep 6, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Sep 6, 2023

New changes are detected. LGTM label has been removed.

@parametalol parametalol temporarily deployed to development September 6, 2023 08:29 — with GitHub Actions Inactive
@parametalol parametalol temporarily deployed to development September 6, 2023 08:29 — with GitHub Actions Inactive
@parametalol parametalol temporarily deployed to development September 6, 2023 08:29 — with GitHub Actions Inactive
@openshift-merge-robot
Copy link
Collaborator

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@kovayur kovayur closed this Jan 17, 2024
@kovayur kovayur deleted the 0x656b694d/e2e-quota-type branch January 17, 2024 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.