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

feat(jdp): add env vars to enable/configure JDP #804

Merged
merged 5 commits into from
Feb 2, 2022

Conversation

andrewazores
Copy link
Member

Fixes #803

@andrewazores andrewazores added the feat New feature or request label Jan 26, 2022
@andrewazores andrewazores requested a review from ebaron January 26, 2022 23:11
@andrewazores andrewazores added feat New feature or request and removed feat New feature or request labels Jan 26, 2022
ebaron
ebaron previously approved these changes Feb 2, 2022
Copy link
Member

@ebaron ebaron left a comment

Choose a reason for hiding this comment

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

Looks good, thanks Andrew! Do you think there's value in the operator explicitly setting this new environment variable to false, even though it's false by default?

@ebaron
Copy link
Member

ebaron commented Feb 2, 2022

I had another thought about this for backporting. Should we keep the default as true in the backport for backwards compatibility?

@andrewazores
Copy link
Member Author

That's a good idea. I'll set the default to true, and I think then the operator should explicitly set it to false.

@andrewazores andrewazores merged commit 5fd17e7 into cryostatio:main Feb 2, 2022
@andrewazores andrewazores deleted the optional-jdp branch February 2, 2022 20:20
mergify bot pushed a commit that referenced this pull request Feb 8, 2022
* feat(jdp): add env var to enable JDP

* feat(jdp): add env vars for JDP addr and port

* enable JDP in development tests

* fix(jdp): broken env var checks

* default JDP to true for backward compatibility

(cherry picked from commit 5fd17e7)

# Conflicts:
#	README.md
#	pom.xml
#	run.sh
andrewazores added a commit that referenced this pull request Feb 9, 2022
* feat(jdp): add env vars to enable/configure JDP (#804)

* feat(jdp): add env var to enable JDP

* feat(jdp): add env vars for JDP addr and port

* enable JDP in development tests

* fix(jdp): broken env var checks

* default JDP to true for backward compatibility

(cherry picked from commit 5fd17e7)

# Conflicts:
#	README.md
#	pom.xml
#	run.sh

* resolve README conflicts

* resolve run.sh conflicts

* resolve pom.xml conflicts

* fixup! resolve README conflicts

* fixup! fixup! resolve README conflicts

* fixup! fixup! fixup! resolve README conflicts

Co-authored-by: Andrew Azores <aazores@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JDP should be optional
2 participants