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

Which spin up flags to support in the shim? #130

Open
kate-goldenring opened this issue May 30, 2024 · 3 comments
Open

Which spin up flags to support in the shim? #130

kate-goldenring opened this issue May 30, 2024 · 3 comments

Comments

@kate-goldenring
Copy link
Collaborator

This issue #52 starts a conversation about supporting spin up flags in the shim. We can ingest these flags from the Container spec args array as follows:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: spin-todo
spec:
  replicas: 1
  selector:
    matchLabels:
      app: spin-todo
  template:
    metadata:
      labels:
        app: spin-todo
    spec:
      runtimeClassName: wasmtime-spin-v2
      containers:
      - name: spin-todo
        image: ttl.sh/spin-kv-checklist:48h
        command: ["/"]
        ports:
        - containerPort: 80
        args: ["--key-value", "foo=true"]

These args can then be processed from the RuntimeContext passed into run_trigger.

My question is which of the flags we want to support. Some such as --listen address are clearly important as noted in #52 . The ones that preseed data, --key-value and --sqlite, could be argued as being uneccessary, especially given that these data sources are likely managed separately of the app lifecycle.

@matsbror
Copy link

matsbror commented May 31, 2024

I had a reason to use --disable-pooling recently on ARM64 and RISCV64 devices with smaller virtual memory address sizes so that would be nice (although this particular issue has been resolved automatically in spin now).

@matsbror
Copy link

matsbror commented Jun 6, 2024

Is the --listen flag supported in the current implementation?

@kate-goldenring
Copy link
Collaborator Author

@matsbror, no. None of the flags are currently supported.

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

No branches or pull requests

2 participants