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

Document standalone and in quarkus configuration #300

Merged
merged 2 commits into from
May 8, 2022

Conversation

cescoffier
Copy link
Contributor

Update the quarkus guide
Show both standalone and in quarkus configuration for all service discovery and load balancer providers

Update the quarkus guide
Show both standalone and in quarkus configuration for all service discovery and load balancer providers
```
=== "stork standalone"
```properties
stork.my-service.load-balancer=acme-load-balancer
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
stork.my-service.load-balancer=acme-load-balancer
stork.my-service.load-balancer.type=acme-load-balancer


=== "stork in quarkus"
```properties
quarkus.stork.my-service.load-balancer=acme-load-balancer
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
quarkus.stork.my-service.load-balancer=acme-load-balancer
quarkus.stork.my-service.load-balancer.type=acme-load-balancer

docs/load-balancer/custom-load-balancer.md Show resolved Hide resolved
docs/load-balancer/custom-load-balancer.md Show resolved Hide resolved

=== "stork standalone"
```properties
stork.my-service.service-discovery=...
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
stork.my-service.service-discovery=...
stork.my-service.service-discovery.type=...


=== "stork in quarkus"
```properties
quarkus.stork.my-service.service-discovery=acme
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
quarkus.stork.my-service.service-discovery=acme
quarkus.stork.my-service.service-discovery.type=acme

```
=== "stork standalone"
```properties
stork.my-service.service-discovery=eureka
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
stork.my-service.service-discovery=eureka
stork.my-service.service-discovery.type=eureka


=== "stork in quarkus"
```properties
quarkus.stork.my-service.service-discovery=eureka
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
quarkus.stork.my-service.service-discovery=eureka
quarkus.stork.my-service.service-discovery.type=eureka

```
=== "stork standalone"
```properties
stork.my-service.service-discovery=kubernetes
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
stork.my-service.service-discovery=kubernetes
stork.my-service.service-discovery.type=kubernetes


=== "stork in quarkus"
```properties
quarkus.stork.my-service.service-discovery=kubernetes
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
quarkus.stork.my-service.service-discovery=kubernetes
quarkus.stork.my-service.service-discovery.type=kubernetes

@cescoffier cescoffier merged commit b36b2c1 into smallrye:main May 8, 2022
@cescoffier cescoffier added this to the 1.2.0 milestone May 8, 2022
@cescoffier cescoffier added the documentation Improvements or additions to documentation label May 8, 2022
@cescoffier cescoffier deleted the update-quarkus-configuration branch June 29, 2022 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants