adding proposal for dockerfile components #3
Conversation
This file contains 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
Signed-off-by: Michael Hoang <[email protected]>
This was referenced Jun 30, 2023
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: elsony, mike-hoang The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
thepetk
referenced
this pull request
in thepetk/devfile-alizer
Jul 6, 2023
Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]>
thepetk
added a commit
that referenced
this pull request
Jul 6, 2023
* Create dependabot.yml Signed-off-by: thepetk <[email protected]> * Update dependabot.yml Signed-off-by: thepetk <[email protected]> * Update realease.yaml (#2) Signed-off-by: thepetk <[email protected]> * adding proposal for dockerfile components (#3) Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Merge all test resources (#6) * Update test paths Signed-off-by: thepetk <[email protected]> * Update angular test resource Signed-off-by: thepetk <[email protected]> * Remove projectAngularjs Signed-off-by: thepetk <[email protected]> * Update containerfile test Signed-off-by: thepetk <[email protected]> * Update django test resources Signed-off-by: thepetk <[email protected]> * Update docker compose test resources Signed-off-by: thepetk <[email protected]> * Update tests for docker compose with ports Signed-off-by: thepetk <[email protected]> * Update test project dockerfile Signed-off-by: thepetk <[email protected]> * Update express js port tests Signed-off-by: thepetk <[email protected]> * Update flask port tests Signed-off-by: thepetk <[email protected]> * Update golang test resources Signed-off-by: thepetk <[email protected]> * Update jboss test resources Signed-off-by: thepetk <[email protected]> * Update laravel test resources Signed-off-by: thepetk <[email protected]> * Update test micronaut resources Signed-off-by: thepetk <[email protected]> * Update container docker file nested tests Signed-off-by: thepetk <[email protected]> * Update nuxt and next js tests Signed-off-by: thepetk <[email protected]> * Update port test for quarkus Signed-off-by: thepetk <[email protected]> * Update reactjs tests Signed-off-by: thepetk <[email protected]> * Remove port test project quarkus Signed-off-by: thepetk <[email protected]> * Update rest of port tests Signed-off-by: thepetk <[email protected]> * Fix issue with ip host go format Signed-off-by: thepetk <[email protected]> * Update test resources Signed-off-by: thepetk <[email protected]> * Finalize new component detection format Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> * Run tidy Signed-off-by: thepetk <[email protected]> * Update devfile_recognizer and models Signed-off-by: thepetk <[email protected]> * Update cli Signed-off-by: thepetk <[email protected]> * Update docs Signed-off-by: thepetk <[email protected]> * Add test cases for versions cli args Signed-off-by: thepetk <[email protected]> * Fix sec alert Signed-off-by: thepetk <[email protected]> * Fix typo Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> Signed-off-by: Michael Hoang <[email protected]> Co-authored-by: Michael Hoang <[email protected]>
thepetk
referenced
this pull request
in thepetk/devfile-alizer
Jul 6, 2023
Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]>
thepetk
added a commit
that referenced
this pull request
Aug 1, 2023
* Add binaries to every new release (#237) * Add release yaml to workflows Signed-off-by: thepetk <[email protected]> * Remove autogeneration of release notes Signed-off-by: thepetk <[email protected]> * Update readme Signed-off-by: thepetk <[email protected]> * Update Readme.md Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> * flattening go dir (#1) Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Update realease.yaml (#2) Signed-off-by: thepetk <[email protected]> * adding proposal for dockerfile components (#3) Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Merge all test resources (#6) * Update test paths Signed-off-by: thepetk <[email protected]> * Update angular test resource Signed-off-by: thepetk <[email protected]> * Remove projectAngularjs Signed-off-by: thepetk <[email protected]> * Update containerfile test Signed-off-by: thepetk <[email protected]> * Update django test resources Signed-off-by: thepetk <[email protected]> * Update docker compose test resources Signed-off-by: thepetk <[email protected]> * Update tests for docker compose with ports Signed-off-by: thepetk <[email protected]> * Update test project dockerfile Signed-off-by: thepetk <[email protected]> * Update express js port tests Signed-off-by: thepetk <[email protected]> * Update flask port tests Signed-off-by: thepetk <[email protected]> * Update golang test resources Signed-off-by: thepetk <[email protected]> * Update jboss test resources Signed-off-by: thepetk <[email protected]> * Update laravel test resources Signed-off-by: thepetk <[email protected]> * Update test micronaut resources Signed-off-by: thepetk <[email protected]> * Update container docker file nested tests Signed-off-by: thepetk <[email protected]> * Update nuxt and next js tests Signed-off-by: thepetk <[email protected]> * Update port test for quarkus Signed-off-by: thepetk <[email protected]> * Update reactjs tests Signed-off-by: thepetk <[email protected]> * Remove port test project quarkus Signed-off-by: thepetk <[email protected]> * Update rest of port tests Signed-off-by: thepetk <[email protected]> * Fix issue with ip host go format Signed-off-by: thepetk <[email protected]> * Update test resources Signed-off-by: thepetk <[email protected]> * Finalize new component detection format Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> * Add min max cli args (#5) * Create dependabot.yml Signed-off-by: thepetk <[email protected]> * Update dependabot.yml Signed-off-by: thepetk <[email protected]> * Update realease.yaml (#2) Signed-off-by: thepetk <[email protected]> * adding proposal for dockerfile components (#3) Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Merge all test resources (#6) * Update test paths Signed-off-by: thepetk <[email protected]> * Update angular test resource Signed-off-by: thepetk <[email protected]> * Remove projectAngularjs Signed-off-by: thepetk <[email protected]> * Update containerfile test Signed-off-by: thepetk <[email protected]> * Update django test resources Signed-off-by: thepetk <[email protected]> * Update docker compose test resources Signed-off-by: thepetk <[email protected]> * Update tests for docker compose with ports Signed-off-by: thepetk <[email protected]> * Update test project dockerfile Signed-off-by: thepetk <[email protected]> * Update express js port tests Signed-off-by: thepetk <[email protected]> * Update flask port tests Signed-off-by: thepetk <[email protected]> * Update golang test resources Signed-off-by: thepetk <[email protected]> * Update jboss test resources Signed-off-by: thepetk <[email protected]> * Update laravel test resources Signed-off-by: thepetk <[email protected]> * Update test micronaut resources Signed-off-by: thepetk <[email protected]> * Update container docker file nested tests Signed-off-by: thepetk <[email protected]> * Update nuxt and next js tests Signed-off-by: thepetk <[email protected]> * Update port test for quarkus Signed-off-by: thepetk <[email protected]> * Update reactjs tests Signed-off-by: thepetk <[email protected]> * Remove port test project quarkus Signed-off-by: thepetk <[email protected]> * Update rest of port tests Signed-off-by: thepetk <[email protected]> * Fix issue with ip host go format Signed-off-by: thepetk <[email protected]> * Update test resources Signed-off-by: thepetk <[email protected]> * Finalize new component detection format Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> * Run tidy Signed-off-by: thepetk <[email protected]> * Update devfile_recognizer and models Signed-off-by: thepetk <[email protected]> * Update cli Signed-off-by: thepetk <[email protected]> * Update docs Signed-off-by: thepetk <[email protected]> * Add test cases for versions cli args Signed-off-by: thepetk <[email protected]> * Fix sec alert Signed-off-by: thepetk <[email protected]> * Fix typo Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> Signed-off-by: Michael Hoang <[email protected]> Co-authored-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Remove dependabot (#10) Signed-off-by: [email protected] Signed-off-by: [email protected] Signed-off-by: thepetk <[email protected]> * Minor update on devfiles versioning (#11) * Remove dependabot Signed-off-by: [email protected] Signed-off-by: thepetk <[email protected]> * Add versions to alizer devfile response Signed-off-by: thepetk <[email protected]> * Update readme.md Signed-off-by: thepetk <[email protected]> * Update naming and devfile models in the proposal Signed-off-by: thepetk <[email protected]> * Update code naming Signed-off-by: thepetk <[email protected]> * Update tests after renaming Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: [email protected] Signed-off-by: thepetk <[email protected]> * fixing gosec alerts (#12) Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Add test coverage check to CI.yaml (#13) * Add test coverage workflow Signed-off-by: thepetk <[email protected]> * Update ci.yaml Signed-off-by: thepetk <[email protected]> * Add separate check for code coverage Signed-off-by: thepetk <[email protected]> * Move code report in ci file Signed-off-by: thepetk <[email protected]> * Add .codecov.yaml Signed-off-by: thepetk <[email protected]> * Update workflow Signed-off-by: thepetk <[email protected]> * Bump up setup-go Signed-off-by:thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> * adding support for dockerfile components (#14) Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: thepetk <[email protected]> * Add test coverage workflow Signed-off-by: thepetk <[email protected]> * Update ci.yaml Signed-off-by: thepetk <[email protected]> * Add separate check for code coverage Signed-off-by: thepetk <[email protected]> * Move code report in ci file Signed-off-by: thepetk <[email protected]> * Update workflow Signed-off-by: thepetk <[email protected]> * Bump up setup-go Signed-off-by:thepetk <[email protected]> Signed-off-by: thepetk <[email protected]> * Make DownloadDevFileTypesFromRegistry public Signed-off-by: thepetk <[email protected]> * Add devfile.yaml schema Signed-off-by: thepetk <[email protected]> * Add go script for generating registry entries json Signed-off-by: thepetk <[email protected]> * Implement nightly run script and workflow Signed-off-by: thepetk <[email protected]> * Update go mod Signed-off-by: thepetk <[email protected]> * Add new schedule to registry check signed-off-by: thepetk <[email protected]> Signed-off-by: thepetk <[email protected]> * Update workflow name Signed-off-by: thepetk <[email protected]> * Update go mod Signed-off-by: thepetk <[email protected]> * Update funcs in order to be mockable Signed-off-by: thepetk <[email protected]> * Move devfile_recognizer_test.go to recognizer dir Signed-off-by: thepetk <[email protected]> * Update docstring of script Signed-off-by: thepetk <[email protected]> * Add tests for check_registry.go Signed-off-by: thepetk <[email protected]> * Fix test paths Signed-off-by: thepetk <[email protected]> * Remove unecessary logging Signed-off-by: thepetk <[email protected]> * Remove binary Signed-off-by: thepetk <[email protected]> * Use make build instead of go command Signed-off-by: thepetk <[email protected]> * Further fixes on the workflow Signed-off-by: thepetk <[email protected]> --------- Signed-off-by: thepetk <[email protected]> Signed-off-by: Michael Hoang <[email protected]> Signed-off-by: [email protected] Co-authored-by: Michael Hoang <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What does this PR do?
Proposal for improving port detection logic for cases where the dockerfiles are not found in a component.
PR moved from redhat-developer/alizer#242
Which issue(s) does this PR fix
fixes devfile/api#1160
PR acceptance criteria
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.
Unit/Functional tests
Documentation
How to test changes / Special notes to the reviewer