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

Improve the UX around bindings #67

Open
3 tasks
dmikusa opened this issue Jul 19, 2021 · 0 comments
Open
3 tasks

Improve the UX around bindings #67

dmikusa opened this issue Jul 19, 2021 · 0 comments
Labels
semver:patch A change requiring a patch version bump type:enhancement A general enhancement

Comments

@dmikusa
Copy link
Contributor

dmikusa commented Jul 19, 2021

What happened?

It's super common for users to attach a binding at build time to pack build, but then forget or not understand that it's also required to have the binding present at runtime. When run without the binding at runtime, the service will typically not work because it is missing credentials and other critical info.

  • What were you attempting to do?

  • What did you expect to happen?

What I think would be reasonable is for libpak to log binding information, like it does with configuration information (the ConfigurationResolver.

Obviously, we wouldn't log the actual value, since that is sensitive information, but we could log the bindings that are visible and perhaps the binding keys that are present, that way a user can see they have bindings set up correctly (getting this set up is tricky).

We can also log a message like, "Buidlpacks do not store bindings in generated images. Please remember to also mount your bindings when you actually run your container images.". Perhaps even link to some docs that explains it more.

  • What was the actual behavior? Please provide log output, if possible.

Nothing. Bindings will be present and may or may not be used. It's confusing and difficult to debug when there are issues.

Checklist

  • I have included log output.
  • The log output includes an error message.
  • I have included steps for reproduction.
@dmikusa dmikusa added semver:patch A change requiring a patch version bump type:enhancement A general enhancement labels Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch A change requiring a patch version bump type:enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

1 participant