Skip to content

Conversation

@romartin
Copy link
Contributor

@romartin romartin commented Dec 21, 2025

Jira Issue: https://issues.redhat.com/browse/AAP-58797

Description

Enabled chatbot authentication.

Testing

Tested operator locally against an AAP instance, in a CRC cluster.

Production deployment

  • This code change is ready for production on its own
  • This PR should be released in 2.4 (cherry-pick should be created)
  • This PR should be released in 2.5 (cherry-pick should be created)
  • This PR should be released in 2.6 (cherry-pick should be created)
  • This code change requires the following considerations before going to production:

@romartin romartin requested a review from ldjebran December 21, 2025 21:49
@ldjebran ldjebran force-pushed the AAP-58797-chatbot-auth branch from a86344f to 238dd91 Compare December 22, 2025 08:22
ldjebran
ldjebran previously approved these changes Dec 22, 2025
Copy link
Contributor

@ldjebran ldjebran left a comment

Choose a reason for hiding this comment

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

LGTM

@ldjebran ldjebran force-pushed the AAP-58797-chatbot-auth branch from 238dd91 to 5be99d2 Compare December 22, 2025 09:47
@ldjebran
Copy link
Contributor

rebased

@ldjebran ldjebran force-pushed the AAP-58797-chatbot-auth branch from 5be99d2 to e39d78f Compare December 22, 2025 14:12
@ldjebran
Copy link
Contributor

rebased

@manstis
Copy link
Contributor

manstis commented Dec 23, 2025

Go to bed 😅

@romartin
Copy link
Contributor Author

romartin commented Dec 23, 2025

Getting crazy @ldjebran

I fixed this operator PR and tested locally. All good, I see the chatbot api key secret and the ENVs, in both wisdom and chatbot containers. Chatbot configuration also OK.

Soooo WHY the f*** the tests are failing??? I can't see still see it :(

Also tested the wisdom status endpoint and I see chatbot endpoints are OK:

{
  "timestamp": "2025-12-23T23:35:59.159788",
  "version": "latest 1.0.202512122233",
  "git_commit": "ca71bf458de906bebc35a31a54e728ba23f9836d",
  "deployed_region": "unknown",
  "status": "error",
  "dependencies": [
    {
      "name": "secret-manager",
      "status": "disabled",
      "time_taken": 0.015
    },
    {
      "name": "authorization",
      "status": "disabled",
      "time_taken": 0.008
    },
    {
      "name": "db",
      "status": "ok",
      "time_taken": 14.801
    },
    {
      "name": "chatbot-service",
      "status": {
        "provider": "http",
        "models": "ok"
      },
      "time_taken": 888.055
    },
    {
      "name": "model-server",
      "status": {
        "provider": "wca",
        "tokens": "unavailable: An error occurred",
        "models": "unavailable: An error occurred"
      },
      "time_taken": 1084.289
    },
    {
      "name": "content-match",
      "status": {
        "provider": "wca",
        "models": "skipped"
      },
      "time_taken": 0.012
    },
    {
      "name": "playbook-explanation",
      "status": {
        "provider": "wca",
        "models": "skipped"
      },
      "time_taken": 0.007
    },
    {
      "name": "playbook-generation",
      "status": {
        "provider": "wca",
        "models": "skipped"
      },
      "time_taken": 0.004
    },
    {
      "name": "role-explanation",
      "status": {
        "provider": "wca",
        "models": "skipped"
      },
      "time_taken": 0.005
    },
    {
      "name": "role-generation",
      "status": {
        "provider": "wca",
        "models": "skipped"
      },
      "time_taken": 0.005
    },
    {
      "name": "streaming-chatbot-service",
      "status": {
        "provider": "http",
        "models": "ok"
      },
      "time_taken": 884.649
    }
  ]
}

@romartin romartin force-pushed the AAP-58797-chatbot-auth branch 5 times, most recently from 5926240 to 770affe Compare December 24, 2025 01:34
@romartin romartin force-pushed the AAP-58797-chatbot-auth branch from 770affe to ddf55bd Compare December 24, 2025 01:48
@romartin romartin enabled auto-merge (squash) December 24, 2025 02:00
@romartin
Copy link
Contributor Author

@ldjebran @jameswnl Ready for review/approval please!! Tested on local and CI all green. Thanks!

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