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

Log query requests and query responses #31

Merged
merged 2 commits into from
Feb 16, 2024
Merged

Log query requests and query responses #31

merged 2 commits into from
Feb 16, 2024

Conversation

jzonthemtn
Copy link
Collaborator

@jzonthemtn jzonthemtn commented Feb 16, 2024

For #1 , here's an example of how the query requests and query responses are indexed with the queryResponseHitIds, queryResponseId, queryId, and the query itself.

These are queries against the store itself so that's why the index is .awesome_queries.

{
  "took": 791,
  "timed_out": false,
  "_shards": {
    "total": 1,
    "successful": 1,
    "skipped": 0,
    "failed": 0
  },
  "hits": {
    "total": {
      "value": 36,
      "relation": "eq"
    },
    "max_score": 1,
    "hits": [
      {
        "_index": ".awesome_queries",
        "_id": "29hrso0BJblqvzTXX2AW",
        "_score": 1,
        "_source": {
          "queryResponseHitIds": [
            "0",
            "1",
            "2",
            "3",
            "4",
            "5",
            "6",
            "7",
            "8",
            "9"
          ],
          "queryResponseId": "fc596fbd-79e8-440d-8607-f45db7df915c",
          "query": "{}",
          "queryId": "b3120e62-35bd-46b4-b36b-4ed4e574b1d3"
        }
      },
      {
        "_index": ".awesome_queries",
        "_id": "2dhrso0BJblqvzTXIGCX",
        "_score": 1,
        "_source": {
          "queryResponseHitIds": [
            "0",
            "1"
          ],
          "queryResponseId": "d4a21117-5a8d-42f6-af92-c301016d1c82",
          "query": "{}",
          "queryId": "28b2e0bc-dc75-4620-8c2b-5aa466cafe4b"
        }
      }
    ]
  }
}

@jzonthemtn jzonthemtn changed the title Log query requests and query responses #1 Log query requests and query responses Feb 16, 2024
@jzonthemtn jzonthemtn changed the title #1 Log query requests and query responses Log query requests and query responses Feb 16, 2024
@RasonJ RasonJ self-requested a review February 16, 2024 15:10
@epugh
Copy link
Member

epugh commented Feb 16, 2024

I wonder if we need the equivalent of "techproducts" to make it clearer what is going on? So less "self referential" ;-). THis is cool to see...

@jzonthemtn
Copy link
Collaborator Author

W

I wonder if we need the equivalent of "techproducts" to make it clearer what is going on? So less "self referential" ;-). THis is cool to see...

We definitely do. If we use it with Chrous we'll have that. Maybe I can get a small subset indexed in these containers for simple demos/testing.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How could we align this config with the dashboard? It's only slightly annoying each time I compile that it asks for the OS version.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should starting adding basic auth, so that we can assume at least some security.

@jzonthemtn jzonthemtn merged commit 51d6388 into main Feb 16, 2024
1 check passed
@jzonthemtn jzonthemtn deleted the log-searches branch February 16, 2024 17:23
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.

3 participants