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

[bug] pouch inspect -f {{.ExecIDs}} returned error #2697

Closed
zjumoon01 opened this issue Jan 28, 2019 · 4 comments · Fixed by #2699
Closed

[bug] pouch inspect -f {{.ExecIDs}} returned error #2697

zjumoon01 opened this issue Jan 28, 2019 · 4 comments · Fixed by #2699
Assignees
Labels
areas/test kind/bug This is bug report for project

Comments

@zjumoon01
Copy link
Contributor

Ⅰ. Issue Description

pouch inspect -f {{.ExecIDs}} is not supported.

#pouch inspect -f {{.ExecIDs}} test-lifecycle-exec-01251548410259
Error: Template parsing error: template: :1:2: executing "" at <.ExecIDs>: can't evaluate field ExecIDs in type *main.inspectContainerJSON

Ⅱ. Describe what happened

It returned error.

Ⅲ. Describe what you expected to happen

It should return its ExecIDs.

Ⅳ. How to reproduce it (as minimally and precisely as possible)

  1. start a container
  2. pouch exec into the container
  3. start a new terminal, and pouch inspect -f {{.ExecIDs}}

Ⅴ. Anything else we need to know?

In fact, {{.ExecIds}} is supported. But "ExecIds" is a variable in code , not json string.
Pouch should support ExecIDs, not ExecIds

Ⅵ. Environment:

  • pouch version (use pouch version): 1.2.0
  • OS (e.g. from /etc/os-release):
  • Kernel (e.g. uname -a): 4.9
  • Install tools:
  • Others:
@pouchrobot pouchrobot added areas/test kind/bug This is bug report for project labels Jan 28, 2019
@HusterWan
Copy link
Contributor

You should use pouch inspect -f {{.ExecIds}}. But we truly not support inspect execids now, I will create PR to fix it.

@HusterWan
Copy link
Contributor

assigin @HusterWan

@fuweid
Copy link
Contributor

fuweid commented Jan 29, 2019

@zjumoon01 could you share your usage here? the execID seems useless thing I guess. Thanks

@zjumoon01
Copy link
Contributor Author

@zjumoon01 could you share your usage here? the execID seems useless thing I guess. Thanks

It is an ops method. Sometimes "exec" is not exited correctly because some bug on docker environment. So inspecting ExecIDs is useful to debug. It will be useful in pouch debug.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
areas/test kind/bug This is bug report for project
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants