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

Adaptive Questionnaire path-based routing #219

Merged
merged 3 commits into from
Apr 6, 2023
Merged

Adaptive Questionnaire path-based routing #219

merged 3 commits into from
Apr 6, 2023

Conversation

c0c0n3
Copy link
Owner

@c0c0n3 c0c0n3 commented Apr 6, 2023

This PR improves the basic Adaptive Questionnaire deployment of #217.

@c0c0n3 c0c0n3 self-assigned this Apr 6, 2023
@c0c0n3 c0c0n3 added the AI / app services Platform application services label Apr 6, 2023
@c0c0n3 c0c0n3 merged commit 9c28e5f into main Apr 6, 2023
@c0c0n3 c0c0n3 deleted the aq-2 branch April 6, 2023 10:13
@c0c0n3
Copy link
Owner Author

c0c0n3 commented Apr 6, 2023

Demo

  • Log into ArgoCD (https://kitt4sme.collab-cloud.eu/argocd) and check out the new Adaptive Questionnaire app.
  • Browse to the Adaptive Questionnaire home page: https://kitt4sme.collab-cloud.eu/aq
  • Hit the "Go to Demo" button.
  • Take the Alloy survey.
  • Check the data in the Postgres DB. (Replace * with the Adaptive Questionnaire DB user's password.)
    • kubectl port-forward svc/postgres 5432:5432
    • psql postgres://ada:*@localhost/adaptive
    • \dt
    • select * from "Answer";

@c0c0n3
Copy link
Owner Author

c0c0n3 commented Apr 6, 2023

@vcutrona please test thoroughly, we still have time to squash any bugs before review day. W/r/t the previous version deployed by #218, I've found these issues:

@karikolehmainen thanks to this PR we don't need port 8443 anymore. But if you could please still keep it open just in case we've got to rollback.

@vcutrona
Copy link
Collaborator

vcutrona commented Apr 11, 2023

@c0c0n3 I tested the AdapQuest deployment.
I experienced the same issues as you, but in addition, the Alloy demo does not provide the questionnaire we need in K4S (it is a different survey).
The deployment should come with an Excel file that contains the right set of questions. I can provide you with the Excel file.

Also, it seems the authentication with Keycloak is missing in this version/deployment.

@c0c0n3
Copy link
Owner Author

c0c0n3 commented Apr 13, 2023

@vcutrona thanks for testing!

provide the questionnaire we need in K4S

yes, this is a demo rollout, when you guys are ready with the actual Kitt4sme questionnaire we'll configure AQ to use that questionnaire instead of the demo ones---see notes in #217 about configuration.

authentication with Keycloak is missing in this version/deployment.

Correct, see #218 and #217 about it :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI / app services Platform application services
Projects
Status: 🚀 Released
Development

Successfully merging this pull request may close these issues.

2 participants