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

Revise docs for using remote workers #342

Merged
merged 2 commits into from
Jan 27, 2025
Merged

Conversation

crajcan
Copy link
Contributor

@crajcan crajcan commented Jan 24, 2025

It seems like a lot has changed since this was written. Everything is stable for Infinyon Cloud workers but we need to verify that what this says actually works for externally hosted remote workers. These updates are written to reflect how I think it is supposed to work

@crajcan crajcan marked this pull request as draft January 24, 2025 23:30
Copy link

cloudflare-workers-and-pages bot commented Jan 24, 2025

Deploying fluvio-docs-nightly with  Cloudflare Pages  Cloudflare Pages

Latest commit: 780c93e
Status: ✅  Deploy successful!
Preview URL: https://bf38d315.fluvio-docs-15u.pages.dev
Branch Preview URL: https://carson-finish-iterating-depl.fluvio-docs-15u.pages.dev

View logs

@crajcan crajcan force-pushed the carson/finish-iterating-deploy branch from 74d2b2a to 70b1f44 Compare January 26, 2025 23:52
@crajcan
Copy link
Contributor Author

crajcan commented Jan 26, 2025

Issues found:

  1. sdf worker create other created a local host worker even though fluvio profile was pointing at fluvio cloud
  2. sdf worker list --all list all workers on all clusters, it looks like --all was originally intended to show the unregistered workers on a cluster in addition to the registered workers.

```

### Managing workers
To view workers that have been provisoned by someone else, you can use `sdf worker list --all`.
Copy link
Contributor Author

@crajcan crajcan Jan 26, 2025

Choose a reason for hiding this comment

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

Need to check if --all is also needed when the same user that created the worker is trying to register it, or if it is already visible.

@crajcan crajcan marked this pull request as ready for review January 27, 2025 00:06
@crajcan crajcan changed the title WIP Revise docs for using remote workers Jan 27, 2025
@crajcan crajcan requested review from sehz and morenol January 27, 2025 00:12
@crajcan crajcan merged commit aa7ca7c into main Jan 27, 2025
2 checks passed
@crajcan crajcan deleted the carson/finish-iterating-deploy branch January 27, 2025 03:56
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