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

Fish 7127 unable to create jvm option #6257

Merged

Conversation

luiseufrasio
Copy link
Contributor

@luiseufrasio luiseufrasio commented Apr 12, 2023

Unable to create JVM option using Web UI

Description

Getting Exception “JVM option${ } already exists in the configuration” upon creating JVM option using WEB UI.

Important Info

Steps

  1. mvn clean install -T 14 -DskipTests
  2. .\appserver\distributions\payara\target\stage\payara6\bin\asadmin start-domain -v
  3. Navigate to Payara -> Server config-> JVM Settings -> Jvm Options Tab
  4. Click on "Add Jvm Option" button
  5. Type: -Dhttps.nonProxyHosts="localhost|127.0.0.1|10...|.fmp-fbz.fgov.be|.fedris.be|.faofat.fgov.be"
  6. Click on Save button
  7. Click on "Add Jvm Option" button
  8. Type: -Dhttp.nonProxyHosts="localhost|127.0.0.1|10...|.fmp-fbz.fgov.be|.fedris.be|.faofat.fgov.be"
  9. Click on Save button

Testing Performed

  • Quicklook
  • Samples

Testing Environment

Zulu JDK 11.0.11 on Windows 11 with Maven 3.8.4

@luiseufrasio
Copy link
Contributor Author

Jenkins test please

Copy link
Member

@Pandrex247 Pandrex247 left a comment

Choose a reason for hiding this comment

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

Not sure this quite works as expected.

When entering the options listed on the JIRA issue it ends up cutting off the closing '"'

@luiseufrasio luiseufrasio requested a review from Pandrex247 April 13, 2023 10:24
@luiseufrasio luiseufrasio merged commit 7649bdd into payara:master Apr 13, 2023
@luiseufrasio luiseufrasio deleted the FISH-7127-unable-to-create-jvm-option branch April 13, 2023 12:18
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