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

CASMCMS-9225: Move CFS client to new paradigm #402

Open
wants to merge 3 commits into
base: casmcms-9225-06-bss-client
Choose a base branch
from

Conversation

mharding-hpe
Copy link
Contributor

@mharding-hpe mharding-hpe commented Dec 17, 2024

CASMCMS-9225 as a whole involves changes to a number of different files in BOS. In order to aid with review, I'm breaking the overall thing up into smaller PRs. Each will be built on top of each other, and will be merging into the main PR branch. Only once each sub-PR has been approved and merged will I merge that branch into develop.
Full list of sub-PRs:

  1. Added basic paging ability for GET requests to list components
  2. Create generic endpoint classes
  3. Create generic API client class
  4. Create ApiClients class and provide it to BOS operators inside a context manager
  5. Move PCS client to new paradigm
  6. Move BSS client to new paradigm
  7. Move CFS client to new paradigm
  8. Move BOS client to new paradigm
  9. Move IMS client to new paradigm
  10. Move HSM client to new paradigm
  11. CHANGELOG update, linting, update utils

This PR is the same as the one for PCS, but for CFS instead. The only noteworthy differences are:

  1. With CFS, there is one filter that makes a CFS call. This PR modifies that filter so that it accepts a CFS client as an argument, modifies the base operator to include a method to provide this filter, using its built-in CFS client, then modifies the impacted operators so they use this filter instead.
  2. The CFS API client uses the cfs_read_timeout option to initialize its requests session.

@mharding-hpe mharding-hpe requested a review from a team as a code owner December 17, 2024 19:30
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-07-cfs-client branch 2 times, most recently from 356be55 to 7bbcd36 Compare December 17, 2024 19:34
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-06-bss-client branch from 5cfcb10 to 116c7ab Compare December 17, 2024 19:37
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-07-cfs-client branch from 7bbcd36 to d8fad17 Compare December 17, 2024 19:37
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-06-bss-client branch from 116c7ab to 92b42a8 Compare December 17, 2024 21:19
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-07-cfs-client branch from 88b9ac7 to d07355a Compare December 17, 2024 21:19
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-06-bss-client branch from 92b42a8 to ba45ffe Compare December 17, 2024 23:27
@mharding-hpe mharding-hpe force-pushed the casmcms-9225-07-cfs-client branch from d07355a to 18c373b Compare December 17, 2024 23:29
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.

2 participants