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

chore: refactor to move ragengine to a central package #671

Merged
merged 7 commits into from
Nov 5, 2024
Merged

Conversation

Fei-Guo
Copy link
Collaborator

@Fei-Guo Fei-Guo commented Nov 5, 2024

This change attempts to move ragengine controller and service code under pkg/ragengine.
Some common codes and labels are moved to pkg/utils and pkg/utils/consts.

There are no functionality changes.

Notes:
This change adds fail faster to e2e test and will keep the failed workspace undeleted (so that the workload pod log can be printed).

@bangqipropel bangqipropel enabled auto-merge (squash) November 5, 2024 17:58
@bangqipropel bangqipropel disabled auto-merge November 5, 2024 17:59
@bangqipropel bangqipropel merged commit b97ab11 into main Nov 5, 2024
6 of 7 checks passed
@bangqipropel bangqipropel deleted the fguo-dev1 branch November 5, 2024 18:30
smritidahal653 pushed a commit to smritidahal653/kaito that referenced this pull request Nov 14, 2024
…#671)

This change attempts to move ragengine controller and service code under
pkg/ragengine.
Some common codes and labels are moved to pkg/utils and
pkg/utils/consts.

There are no functionality changes.

Notes:
This change adds fail faster to e2e test and will keep the failed
workspace undeleted (so that the workload pod log can be printed).

---------

Co-authored-by: Bangqi Zhu <68867834+bangqipropel@users.noreply.github.com>
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.

4 participants