forked from nerc-project/coldfront-plugin-cloud
-
Notifications
You must be signed in to change notification settings - Fork 0
Ops 948/auth kc test #23
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
Open
QuanMPhm
wants to merge
127
commits into
main
Choose a base branch
from
ops_948/auth_kc_test
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Migrating to using upstream keystone oidc plugin
Improving test reliability by removing OIDC plugin from Openstack functional test
Add institution specific code to invoice
- setup.cfg was missing the boto3 library, therefore it wasn't installed in the `coldfront-nerc container`. - String quotes were incorrect in one instance, causing the script to fail. - Auto-complete suggested `add_option` confusing argparse for optparse. Fixed correct usage for argparse, changing the method from add_option to add_argument.
Fix missing dependency, incorrect string quotes and argparse method
Added management command to migrate field of sciences
This switches the PI field in invoices to use email instead.
…_invoices Return PI email in storage invoices
…o-quota openstack: require swift/object/radosgw quotas to be >= 0
The quota value returned from swift's head_account function returns a string not an integer.
…j-quota-comparison validate: fix object store quota value comparison
renamed the attributes with GiB and MiB removing GB and MB
Cost will now be rounded to 2 decimal points Rate will be rounded to 6 decimal points, since the Openshift Gb/hr rates at the time of this commit is $0.000009
Added rounding to Cost and Rate for storage invoice
A one-time command is added which will set the Swift quota of every allocation from GB to GiB. The Swift quota attribute name has been appropriately renamed
…otas Convert Swift quotas from GB to GiB
Added `Cluster Name` column to storage invoice
- Add "Cluster Name" in invoice header - Write Resource Name instead of Resource Type Name Related nerc-project#221
…ader Fix Cluster Name in invoice
Allow direct communication to Openshift Projects API
Allow communication to Openshift Users API
Allow direct communication to Openshift RBAC API
…ations `validate_allocations` will now check if an Openshift allocation does not have a quota value set on either the Coldfront or Openshift side. In this case, it will set the default quota value for the allocation. Due to the complexity of Openstack quotas, this feature is only implemented for Openshift allocations for now.
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4...v5) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
Added timezone awareness to default start and end date arguments. Fixed `s3-bucket-name` cli arguement name.
Fix `calculate_storage_gb_hours`
…ub_actions/actions/checkout-5 Bump actions/checkout from 4 to 5
Implemented feature to add new attributes to existing Openshift allocations
…ct_manager Removed references to openshift-acct-mgt
The addition of IBM Scale storage to OpenShift resources has several implications: - A pre-existing Openshift attribute, `QUOTA_REQUESTS_STORAGE`, was renamed to include "NESE" in its name to avoid confusion with the new IBM storage attribute. - OpenShift resources now have a new attribute `IBM Spectrum Scale Storage Available` ("true" or "false") to indicate IBM Scale storage availability. If "true", new and pre-existing allocations will have a new attribute to indicate IBM storage quota, which defaults to 0 Gb. - Openshift quota key for NESE storage is now `ocs-external-storagecluster-ceph-rbd.storageclass.storage.k8s.io/requests.storage` For now, IBM storage quota is free and not included in the storage invoice.
Closes https://github.com/issues/assigned?issue=CCI-MOC%7Cnerc-rates%7C45. Migrate of our codebases to use the more specific NESE Storage GB Rate rate when referring to storage
Created new allocation attributes for IBM storage quota
Migrate to use NESE Storage GB Rate
237/ibm in invoice
Some containers in the RHOAI application run with 64Mi of memory. The original reason we set these minimum is to prevent people from running containers without any cpu or memory request.
…imum-resources This lowers the minimum resources that a container must run with.
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5 to 6. - [Release notes](https://github.com/actions/setup-python/releases) - [Commits](actions/setup-python@v5...v6) --- updated-dependencies: - dependency-name: actions/setup-python dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
…ub_actions/actions/setup-python-6 Bump actions/setup-python from 5 to 6
Allow validation of allocations with status `Active (Needs Renewal)`
A Keycloak admin client has been added When `activate_allocation` is called, the user is added to a Keycloak group named after the project ID on the remote cluster. If the user does not already exist in Keycloak, the case is ignored for now
2d72d72
to
6f61e46
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.