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

[CLOUD-1867] update datavirt-app-config secret to use new src location #74

Merged
merged 1 commit into from
Feb 8, 2018
Merged

Conversation

osnipso
Copy link
Contributor

@osnipso osnipso commented Aug 30, 2017

Issue: CLOUD-1867

Changes default value for APP_DATADIR (== data). Also, change tests to rely on new default value for APP_DATADIR.
Update datavirt-app-config secret to use new src location (/opt/eap/standalone/data/).
Add a new CTF test for custom APP_DATADIR value.

This PR works in conjunction with #387 in application-templates.
This PR replaces #305 in application-templates.

Please make sure your PR meets following requirements:

  • Pull Request title is properly formatted: [CLOUD-XYA] Subject
  • Pull Request contains link to the JIRA issue
  • Pull Request contains description of the issue
  • Pull request does not include fixes for other issues than the main ticket
  • Attached commits represent unit of work and are properly formatted
  • You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
  • Every commit contains Signed-off-by: Your Name <[email protected]> - use git commit -s

@@ -30,7 +30,6 @@ Feature: Openshift JDV s2i tests
| EXCEL_CONNECTION_JNDI | java:/excel-file |
| EXCEL_PROPERTY_ParentDirectory | /home/jboss/source/data/teiidfiles/excelFiles/ |
| EXCEL_PROPERTY_AllowParentPaths | true |
| APP_DATADIR | data |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add a test that checks for a non-default value for APP_DATADIR?

rcernich
rcernich previously approved these changes Sep 12, 2017
@osnipso osnipso changed the title [CLOUD-1867] update datavirt-app-config secret to use new src location (WIP) [CLOUD-1867] update datavirt-app-config secret to use new src location Oct 5, 2017
Change existing test to rely on new default value for APP_DATADIR (== data)
Add new test for custom APP_DATADIR

Signed-off-by: Osni Oliveira <[email protected]>
@osnipso osnipso changed the title (WIP) [CLOUD-1867] update datavirt-app-config secret to use new src location [CLOUD-1867] update datavirt-app-config secret to use new src location Oct 19, 2017
@rcernich rcernich merged commit d7dd4d9 into jboss-openshift:master Feb 8, 2018
@osnipso osnipso deleted the CLOUD-1867 branch February 12, 2018 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants