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.

Sign up for GitHub

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

Jump to bottom

Run some tests against the high level connection status interface #316

Merged
merged 31 commits into from
Jan 17, 2024
Merged

Run some tests against the high level connection status interface #316

merged 31 commits into from
Jan 17, 2024

Conversation

dehort
Copy link
Contributor

@dehort dehort commented Dec 14, 2023 •
edited
Loading

  • Changed high level connection check to require the identity header as inventory does not support PSKs
  • Fixed order_by and order_how params for inventory client

@dehort dehort changed the title Wow Run some tests against the high level connection status interface Dec 14, 2023
@dehort
Copy link
Contributor Author

dehort commented Jan 10, 2024

/retest

@dehort
Copy link
Contributor Author

dehort commented Jan 11, 2024

I got some data back:

[ { "org_id": "10000", "recipient": "6f93cfc7-5edb-4ebc-bc42-1b6ad398a410", "recipient_type": "directConnect", "sat_id": "", "sat_org_id": "", "status": "connected", "systems": [ "ec9e899b-4e79-434d-a55b-b2965e7095ab" ] } ]

@dehort
Copy link
Contributor Author

dehort commented Jan 11, 2024 •
edited
Loading

Correctly shows system as disconnected:

[ { "org_id": "10000", "recipient": "6f93cfc7-5edb-4ebc-bc42-1b6ad398a410", "recipient_type": "directConnect", "sat_id": "", "sat_org_id": "", "status": "disconnected", "systems": [ "ec9e899b-4e79-434d-a55b-b2965e7095ab" ] } ]

dehort added 20 commits January 12, 2024 09:03
@dehort
Wow
92069b1
@dehort
Testing
fb4ec32
@dehort
Testing
a9e867c
@dehort
Testing
792a8d6
@dehort
Testing - don't require a PSK for the high level connection status
52638dd
@dehort
adding more debug logging
cb9e17d
@dehort
adding more debug logging
cb3eb1a
@dehort
Try to require the identity header and store it in the request context
8baf2f3
@dehort
More debugging
6c19792
@dehort
Testing
e315806
@dehort
TEsting
dbc54dd
@dehort
More
e3754fb
@dehort
More
04d54d4
@dehort
More
43ef217
@dehort
Closer
40d645d
@dehort
ah, ha...how and by are swapped here
0b781dd
@dehort
Trying to figure out if system profile call is working
a5d5611
@dehort
My simple caveman brain can't keep the orderBy vs orderHow straight.
c9af27f 
They are both strings so the compiler can't catch us/help us when these
get reversed.  We could make this separate type defs to help with that.

Looks like orderBy should be first in the arg list...so that's what i'm
trying to go with.
@dehort
Print the response out in human readable form...because i'm a human a…
1ad8503 
…nd i want to read it
@dehort
rebased
163f99a
@dehort
Copy link
Contributor Author

dehort commented Jan 15, 2024

/retest

@dehort
Copy link
Contributor Author

dehort commented Jan 17, 2024

Ugh

@dehort dehort marked this pull request as ready for review January 17, 2024 17:41
@dehort dehort merged commit 7d0700d into RedHatInsights:master Jan 17, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dehort