Skip to content

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

Merged
openshift-merge-robot merged 3 commits intoredhat-developer:mainfrom
rm3l:6274-support-variable-substitution-with-odo-build-images
Apr 27, 2023
Merged

Support variable substitution with --var and --var-file in odo build-images#6776
openshift-merge-robot merged 3 commits intoredhat-developer:mainfrom
rm3l:6274-support-variable-substitution-with-odo-build-images

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
@sonarqubecloud
Copy link

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

Details

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

Details

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