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

Support variable substitution with --var and --var-file in odo build-images #6776

Conversation

rm3l
Copy link
Member

@rm3l rm3l commented Apr 27, 2023

What type of PR is this:

/kind feature

What does this PR do / why we need it:

Which issue(s) this PR fixes:
Fixes #6274

PR acceptance criteria:

How to test changes / Special notes to the reviewer:

You can run odo build-images [--var ... | --var-file ...] using the Devfile used in the tests (tests/examples/source/devfiles/nodejs/devfile-variables.yaml) or any Devfile where image names are variables (like in our Advanced Guides for deploying apps)

@openshift-ci openshift-ci bot added the kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation label Apr 27, 2023
@netlify
Copy link

netlify bot commented Apr 27, 2023

Deploy Preview for odo-docusaurus-preview ready!

Name Link
🔨 Latest commit 5c3dc53
🔍 Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/644a204551be9000086e7ced
😎 Deploy Preview https://deploy-preview-6776--odo-docusaurus-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@openshift-ci openshift-ci bot requested review from anandrkskd and feloy April 27, 2023 07:08
@rm3l rm3l force-pushed the 6274-support-variable-substitution-with-odo-build-images branch from c851d6b to 5c3dc53 Compare April 27, 2023 07:12
@rm3l rm3l requested review from valaparthvi and removed request for anandrkskd April 27, 2023 07:12
@sonarcloud
Copy link

sonarcloud bot commented Apr 27, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

NoCluster Tests on commit 331e8cb finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

OpenShift Unauthenticated Tests on commit 331e8cb finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

Unit Tests on commit 331e8cb finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

Validate Tests on commit 331e8cb finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

Windows Tests (OCP) on commit 331e8cb finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

Kubernetes Tests on commit 331e8cb finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

Kubernetes Docs Tests on commit 5913097 finished successfully.
View logs: TXT HTML

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Apr 27, 2023
@valaparthvi
Copy link
Contributor

valaparthvi commented Apr 27, 2023

/override windows-integration-test/Windows-test

Windows Failure:


Summarizing 4 Failures:
  [FAIL] odo remove binding command tests [AfterEach] when the component with binding is bootstrapped (bindingName=my-nodejs-app-cluster-sample-k8s) when odo dev is running when binding is removed should have led odo dev to delete ServiceBinding from the cluster
  C:/Users/Administrator.ANSIBLE-TEST-VS/3828/tests/helper/helper_cmd_wrapper.go:101
  [FAIL] odo add binding interactive command tests [BeforeEach] when the component is bootstrapped should successfully add binding to the devfile ("lowercase" as naming strategy)
  C:/Users/Administrator.ANSIBLE-TEST-VS/3828/tests/helper/helper_generic.go:58
  [FAIL] odo logs command tests [BeforeEach] when component is created and odo logs is executed when running in both Dev and Deploy mode should successfully show logs of the running component
  C:/Users/Administrator.ANSIBLE-TEST-VS/3828/tests/helper/helper_cmd_wrapper.go:101
  [FAIL] odo describe/list binding command tests [BeforeEach] when creating a component with a binding (service in namespace "") should list the binding without running odo dev ([list])
  C:/Users/Administrator.ANSIBLE-TEST-VS/3828/tests/helper/helper_cmd_wrapper.go:101

Ran 449 of 830 Specs in 1967.082 seconds
FAIL! -- 445 Passed | 4 Failed | 0 Pending | 381 Skipped

All issues relate to network connectivity.

@valaparthvi
Copy link
Contributor

/override windows-integration-test/Windows-test

@openshift-ci
Copy link

openshift-ci bot commented Apr 27, 2023

@valaparthvi: Overrode contexts on behalf of valaparthvi: windows-integration-test/Windows-test

In response to this:

/override windows-integration-test/Windows-test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@odo-robot
Copy link

odo-robot bot commented Apr 27, 2023

OpenShift Tests on commit 331e8cb finished with errors.
View logs: TXT HTML

@rm3l
Copy link
Member Author

rm3l commented Apr 27, 2023

• [FAILED] [471.677 seconds]
odo dev command tests when Starting a PostgreSQL service when creating local files and dir and running odo dev - without metadata.name [AfterEach] when deleting local files and dir and waiting for sync should not list deleted dir and file in container
...
  [odo] I0427 10:47:06.267582   28067 delete.go:94] failed to delete resource "my-nodejs-app-cluster-sample-ocp" (binding.operators.coreos.com.v1alpha1.servicebindings): timeout while waiting for "my-nodejs-app-cluster-sample-ocp" resource to be deleted

  [FAILED] Timed out after 180.000s.
  Expected process to exit.  It did not.
  In [AfterEach] at: /go/odo_1/tests/helper/helper_dev.go:213 @ 04/27/23 10:47:07.106
...
Summarizing 1 Failure:
  [FAIL] odo dev command tests when Starting a PostgreSQL service when creating local files and dir and running odo dev - without metadata.name [AfterEach] when deleting local files and dir and waiting for sync should not list deleted dir and file in container
  /go/odo_1/tests/helper/helper_dev.go:213

Ran 458 of 830 Specs in 2435.435 seconds
FAIL! -- 457 Passed | 1 Failed | 0 Pending | 372 Skipped

Unrelated.

/override OpenShift-Integration-tests/OpenShift-Integration-tests

@openshift-ci
Copy link

openshift-ci bot commented Apr 27, 2023

@rm3l: Overrode contexts on behalf of rm3l: OpenShift-Integration-tests/OpenShift-Integration-tests

In response to this:

• [FAILED] [471.677 seconds]
odo dev command tests when Starting a PostgreSQL service when creating local files and dir and running odo dev - without metadata.name [AfterEach] when deleting local files and dir and waiting for sync should not list deleted dir and file in container
...
 [odo] I0427 10:47:06.267582   28067 delete.go:94] failed to delete resource "my-nodejs-app-cluster-sample-ocp" (binding.operators.coreos.com.v1alpha1.servicebindings): timeout while waiting for "my-nodejs-app-cluster-sample-ocp" resource to be deleted

 [FAILED] Timed out after 180.000s.
 Expected process to exit.  It did not.
 In [AfterEach] at: /go/odo_1/tests/helper/helper_dev.go:213 @ 04/27/23 10:47:07.106
...
Summarizing 1 Failure:
 [FAIL] odo dev command tests when Starting a PostgreSQL service when creating local files and dir and running odo dev - without metadata.name [AfterEach] when deleting local files and dir and waiting for sync should not list deleted dir and file in container
 /go/odo_1/tests/helper/helper_dev.go:213

Ran 458 of 830 Specs in 2435.435 seconds
FAIL! -- 457 Passed | 1 Failed | 0 Pending | 372 Skipped

Unrelated.

/override OpenShift-Integration-tests/OpenShift-Integration-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-merge-robot openshift-merge-robot merged commit 00918c3 into redhat-developer:main Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support variable substitution with odo build-images
3 participants