Skip to content
This repository was archived by the owner on Feb 4, 2025. It is now read-only.

Do Not Merge - Runtimes PoC#123

Draft
Jakub007d wants to merge 12 commits intomasterfrom
poc-runtimes
Draft

Do Not Merge - Runtimes PoC#123
Jakub007d wants to merge 12 commits intomasterfrom
poc-runtimes

Conversation

@Jakub007d
Copy link
Collaborator

Description

The poc-runtimes branch is used for runtimes PoC. The DVO-Extractor was adapted to be able to process the runtimes recommendations. This MR is used to build image in quay so it can be ran in Ephemeral environment.

!!!!DO NOT MERGE!!!!

Fixes # CCXDEV-14539

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Configuration update

Checklist

  • make before_commit passes
  • updated documentation wherever necessary
  • added or modified tests if necessary
  • updated schemas and validators in insights-data-schemas in case of input/output change

@ikerreyes
Copy link
Collaborator

/retest

ikerreyes and others added 4 commits November 28, 2024 16:11
Formatters are not completely independent from rules as they need
to recognize them.
Update formatter to be the one for runtimes
Fixing wrong reference to runtime rules
@ikerreyes ikerreyes marked this pull request as draft December 2, 2024 10:14
@Jakub007d
Copy link
Collaborator Author

/retest

@Jakub007d Jakub007d marked this pull request as ready for review December 3, 2024 08:07
@Jakub007d
Copy link
Collaborator Author

/retest

@Jakub007d Jakub007d marked this pull request as draft December 3, 2024 08:29
@Jakub007d
Copy link
Collaborator Author

/retest

Jakub007d and others added 5 commits December 3, 2024 09:50
Updating reference to rules after rules repo update
Need a new commit to trigger image rebuild as /retest comment won't do becuase the commit has not changed
@ikerreyes
Copy link
Collaborator

/retest

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants