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

Make all pkg applications stateful #17997

Closed
3 tasks done
jsteenb2 opened this issue May 7, 2020 · 0 comments · Fixed by #18279
Closed
3 tasks done

Make all pkg applications stateful #17997

jsteenb2 opened this issue May 7, 2020 · 0 comments · Fixed by #18279
Labels
area/templates influxdb templates and pkger related issues team/user-tools

Comments

@jsteenb2
Copy link
Contributor

jsteenb2 commented May 7, 2020

AC

  • if no stack id is provided for application, then one is created
  • if stack id is provided, it is used for application
  • summary response should return the stackID

note:
it may be worthwhile refactoring the tuples from apply/dry run to be something like

// name is just a stand in, not sure what a name may look like
type PkgDefinition struct {
    StackID influxdb.ID
    Summary Summary
    Diff Diff
}
@jsteenb2 jsteenb2 added the area/templates influxdb templates and pkger related issues label May 26, 2020
jsteenb2 added a commit that referenced this issue May 28, 2020
jsteenb2 added a commit that referenced this issue May 28, 2020
jsteenb2 added a commit that referenced this issue May 28, 2020
if no stack is provided then one will be provided for you.

closes: #17997
jsteenb2 added a commit that referenced this issue May 28, 2020
if no stack is provided then one will be provided for you.

closes: #17997
jsteenb2 added a commit that referenced this issue May 28, 2020
if no stack is provided then one will be provided for you.

closes: #17997
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/templates influxdb templates and pkger related issues team/user-tools
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant