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

LocalPubSubHelper does not work with gcloud beta emulators #990

Closed
mziccard opened this issue May 6, 2016 · 1 comment
Closed

LocalPubSubHelper does not work with gcloud beta emulators #990

mziccard opened this issue May 6, 2016 · 1 comment
Assignees
Labels
api: pubsub Issues related to the Pub/Sub API.

Comments

@mziccard
Copy link
Contributor

mziccard commented May 6, 2016

gcloud's --host-port requires both an host and a port to be provided (in the format host:port). This line must therefore be changed to:

gcloudCommand.add(GCLOUD_CMD_PORT_FLAG + "localhost:" + port);

Also we need to change the blockUntilOutput to be more specific that the simple port number.
gcloud beta emulators pubsub start in fact first logs the underlying command:

Executing: /bin/bash <gcloud-path>/platform/pubsub-emulator/bin/cloud-pubsub-fake --host=localhost --port=<port>

So waiting just for the port to be logged is not enough. I suggest we wait for the whole string:

"Server started, listening on " + port

/cc @garrettjonesgoogle @shinfan

@mziccard mziccard added api: pubsub Issues related to the Pub/Sub API. testing labels May 6, 2016
@mziccard
Copy link
Contributor Author

mziccard commented May 8, 2016

This was closed in #992

@mziccard mziccard closed this as completed May 8, 2016
github-actions bot pushed a commit that referenced this issue Sep 15, 2022
…1.1 (#990)

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:libraries-bom](https://cloud.google.com/java/docs/bom) ([source](https://togithub.com/googleapis/java-cloud-bom)) | `26.1.0` -> `26.1.1` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/compatibility-slim/26.1.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:libraries-bom/26.1.1/confidence-slim/26.1.0)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox. ⚠ **Warning**: custom changes will be lost.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-dialogflow).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4xODQuMiIsInVwZGF0ZWRJblZlciI6IjMyLjE4NC4yIn0=-->
github-actions bot pushed a commit that referenced this issue Oct 4, 2022
…telligence-bom to v2.2.7 (#990)

* chore(deps): update dependency com.google.cloud:google-cloud-video-intelligence-bom to v2.2.7

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Diego Alonso Marquez Palacios <diegomarquezp@google.com>
suztomo pushed a commit that referenced this issue Feb 1, 2023
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [io.grpc:grpc-bom](https://togithub.com/grpc/grpc-java) | `1.50.1` -> `1.50.2` | [![age](https://badges.renovateapi.com/packages/maven/io.grpc:grpc-bom/1.50.2/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/io.grpc:grpc-bom/1.50.2/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/io.grpc:grpc-bom/1.50.2/compatibility-slim/1.50.1)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/io.grpc:grpc-bom/1.50.2/confidence-slim/1.50.1)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>grpc/grpc-java</summary>

### [`v1.50.2`](https://togithub.com/grpc/grpc-java/releases/tag/v1.50.2)

##### Bug fixes

gcp-observability: Supports period(.) in the service name part of regular expression for a fully-qualified method to accept "package.service"

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-core).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzMi4yNDAuNSIsInVwZGF0ZWRJblZlciI6IjMyLjI0MS4xMSJ9-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the Pub/Sub API.
Projects
None yet
Development

No branches or pull requests

1 participant