Skip to content

Commit

Permalink
Merge pull request #418 from xstefank/wfly-13467
Browse files Browse the repository at this point in the history
WFLY-13467 Issues in readme for microprofile-opentracing QS
  • Loading branch information
emmartins committed May 14, 2020
2 parents 48d269f + f5eafbd commit 7bf3923
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 14 deletions.
40 changes: 26 additions & 14 deletions microprofile-opentracing/README.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ The `microprofile-opentracing` quickstart demonstrates the use of the MicroProfi
:standalone-server-type: default
:archiveType: war
:archiveName: {artifactId}
:additional-startup-params: JAEGER_REPORTER_LOG_SPANS=true JAEGER_SAMPLER_TYPE=const JAEGER_SAMPLER_PARAM=1 \

== What is it?

Expand All @@ -26,23 +27,34 @@ https://zipkin.io/[Zipkin]).
In this quickstart, we have a collection of CDI beans and REST endpoints that
expose functionalities of the MicroProfile OpenTracing specification.

// System Requirements
include::../shared-doc/system-requirements.adoc[leveloffset=+1]

// Use of {jbossHomeName}
include::../shared-doc/use-of-jboss-home-name.adoc[leveloffset=+1]

// Start the {productName} Standalone Server
include::../shared-doc/start-the-standalone-server.adoc[leveloffset=+1]

== Solution

We recommend that you follow the instructions in the next sections and create the
application step by step. However, you can go right to the completed example which
is available in this directory.
We recommend that you follow the instructions that
<<creating-new-project, create the application step by step>>. However, you can
also go right to the completed example which is available in this directory.

. Start your {productName} server
// Build and Deploy the Quickstart
include::../shared-doc/build-and-deploy-the-quickstart.adoc[leveloffset=+1]

. Compile the application code and deploy it to {productName}
[source,options="nowrap"]
----
$ mvn clean package wildfly:deploy
----
// Undeploy the Quickstart
include::../shared-doc/undeploy-the-quickstart.adoc[leveloffset=+1]

// Start the {productName} Standalone Server
include::../shared-doc/start-the-standalone-server.adoc[leveloffset=+1]
// Run the Arquillian Tests
include::../shared-doc/run-arquillian-tests.adoc[leveloffset=+1]

// Run the Quickstart in Red Hat CodeReady Studio or Eclipse
include::../shared-doc/run-the-quickstart-in-jboss-developer-studio.adoc[leveloffset=+1]

[[creating-new-project]]
== Creating the Maven Project

[source,options="nowrap"]
Expand Down Expand Up @@ -84,19 +96,19 @@ Now we need to add the following dependencies:

[source,xml]
----
<!-- Import the MicroProfile OpenTracing API, we use provided scope as the API is included in WildFly -->
<!-- Import the MicroProfile OpenTracing API, we use provided scope as the API is included in the server -->
<dependency>
<groupId>org.eclipse.microprofile.opentracing</groupId>
<artifactId>microprofile-opentracing-api</artifactId>
<scope>provided</scope>
</dependency>
<!-- Import the CDI API, we use provided scope as the API is included in WildFly -->
<!-- Import the CDI API, we use provided scope as the API is included in the server -->
<dependency>
<groupId>jakarta.enterprise</groupId>
<artifactId>jakarta.enterprise.cdi-api</artifactId>
<scope>provided</scope>
</dependency>
<!-- Import the Jakarta REST API, we use provided scope as the API is included in WildFly -->
<!-- Import the Jakarta REST API, we use provided scope as the API is included in the server -->
<dependency>
<groupId>org.jboss.spec.javax.ws.rs</groupId>
<artifactId>jboss-jaxrs-api_2.1_spec</artifactId>
Expand Down
8 changes: 8 additions & 0 deletions shared-doc/start-the-standalone-server.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,18 @@ include::define-standalone-server-attributes.adoc[]
. Open a terminal and navigate to the root of the {productName} directory.
. Start the {productName} server with the {serverProfile} by typing the following command.
+
ifdef::additional-startup-params[]
[source,subs="+quotes,attributes+",options="nowrap"]
----
$ __{additional-startup-params}__ __{jbossHomeName}__/bin/standalone.sh {serverArguments}
----
endif::[]
ifndef::additional-startup-params[]
[source,subs="+quotes,attributes+",options="nowrap"]
----
$ __{jbossHomeName}__/bin/standalone.sh {serverArguments}
----
endif::[]
+
NOTE: For Windows, use the `__{jbossHomeName}__\bin\standalone.bat` script.
Expand Down

0 comments on commit 7bf3923

Please sign in to comment.