FLPATH-2749 | Refactor the Plugin code based on new structure to cost…#2274
Closed
asmasarw wants to merge 39 commits intoredhat-developer:mainfrom
Closed
FLPATH-2749 | Refactor the Plugin code based on new structure to cost…#2274asmasarw wants to merge 39 commits intoredhat-developer:mainfrom
asmasarw wants to merge 39 commits intoredhat-developer:mainfrom
Conversation
asmasarw
requested review from
PreetiW,
batzionb,
mareklibra and
ydayagi
as code owners
Missing ChangesetsThe following package(s) are changed by this PR but do not have a changeset:
See CONTRIBUTING.md for more information about how to add changesets. Changed Packages
|
Contributor
|
are we renaming the plugin to Cost Management? |
Contributor
Author
|
yes @ydayagi |
asmasarw
force-pushed
the
feature/refactor-plugin
branch
from
8e1ef40 to
ea44033
Compare
Contributor
PreetiW
requested changes
Feb 12, 2026
Contributor
PreetiW
left a comment
PreetiW
left a comment
There was a problem hiding this comment.
@asmasarw as a part of the card we wanted to refactor the plugin folders as well, can you check the same? at the moment the plugin code structure is around redhat-resource-optimization but since we changed the plugin to cost management I think it will better to refactor each plugin to cost-management which was the scope of the jira. Let me know what you think? if you think it is too risky then we can discuss! 🙌🏻
asmasarw
force-pushed
the
feature/refactor-plugin
branch
from
9438a72 to
ea44033
Compare
…loper#2285) Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…oper#2293) * chores(x2a): refactor router for better maintainability Signed-off-by: Marek Libra <marek.libra@gmail.com> * refactor router tests and increase coverage Signed-off-by: Marek Libra <marek.libra@gmail.com> * Fix tests --------- Signed-off-by: Marek Libra <marek.libra@gmail.com>
redhat-developer#2296) Signed-off-by: Marek Libra <marek.libra@gmail.com>
…-developer#2299) Signed-off-by: Marek Libra <marek.libra@gmail.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
* First draft: Real Job creation * job template is able to git pull * Working full init job including git * init phase coomplete, skeleton for other phases * With fixes tests, yarn full * app-config reset * rbac * reverting aap and rbac to main * prettier * Working Analyze phase + owner reference delete secret * analyze job works * migrate changes * remove callback url call * Working analyze + migration steps * api report * After rebase * Error handling on bash script * Tests error
…veloper#2302) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Signed-off-by: Jon Koops <jonkoops@gmail.com>
…er#2255) * RHINENG-23334 updated docs for Cost Management plugin * chore: updated README.md
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…veloper#2266) * feat(x2a): Enable backend application for collectartifacts This is the initial implementation on the collectArtifacts, there are a few things missing(Marked as TODO), but I think that it'll be good to have a bootstrap already, and improved in different steps like security and events on job finished. Signed-off-by: Eloy Coto <eloy.coto@acalustra.com> * fix(x2a): PR feedback Signed-off-by: Eloy Coto <eloy.coto@acalustra.com> --------- Signed-off-by: Eloy Coto <eloy.coto@acalustra.com>
…to the modules table (redhat-developer#2310) Signed-off-by: Marek Libra <marek.libra@gmail.com>
…edhat-developer#2281) * feat(x2a): renew git repo tokens before submitting run-phase action Signed-off-by: Marek Libra <marek.libra@gmail.com> * review --------- Signed-off-by: Marek Libra <marek.libra@gmail.com>
…t-developer#2312) Signed-off-by: Jon Koops <jonkoops@gmail.com>
* feat(x2a): add project and module status Signed-off-by: Marek Libra <marek.libra@gmail.com> * Add errorDetails to the UI * yarn.lock * review --------- Signed-off-by: Marek Libra <marek.libra@gmail.com>
asmasarw
requested review from
a team,
Eswaraiahsapram,
Lucifergene,
MatousJobanek,
alexeykazakov,
christoph-jerolimov,
ciiay,
debsmita1,
divyanshiGupta,
eloycoto,
gabemontero,
gciavarrini,
its-mitesh-kumar,
johnmcollier,
karthikjeeyar,
logonoff,
mfrancisc,
rajivnathan,
rm3l,
rohitkrai03,
rrosatti,
rsoaresd,
sahil143,
testcara,
thepetk,
xcoulon and
yangcao77
as code owners
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refactor the Plugin code based on new structure to cost-management