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

wsk sdk install docker fails due to missing tarball #74

Closed
tybritten opened this issue Mar 14, 2016 · 12 comments
Closed

wsk sdk install docker fails due to missing tarball #74

tybritten opened this issue Mar 14, 2016 · 12 comments

Comments

@tybritten
Copy link

the command
wsk sdk install docker
looks for the tarball on the API host. it's not being hosted by the API endpoint and the command fails.

@perryibm
Copy link
Contributor

There is a related bug in that the CLI itself is also not hosted at the API host. There are some gaps in both the build and deployment scripts caused by a conflation of developer vs user role. The first step is to make the dockerSkeleton tar ball available in the directory structure after a build and we should have this done soon. The second step is to make sure both tar balls (CLI and docker skeleton) which may be delayed until our migration to a new build/deploy system is done.

@sjfink
Copy link
Contributor

sjfink commented May 18, 2016

@perryibm is there any status update on this?

@perryibm
Copy link
Contributor

@csantanapr is in process of moving this to open.

@csantanapr
Copy link
Member

csantanapr commented May 18, 2016

@perryibm I opened issue #441, the issue is in backlog no takers yet
Based on your #74 (comment)

If the only work remaining is moving the dockerSkeleton to open, then I will close #441 as a duplicate of this one since @vmtyler opened this one first, and he can get notified when this issue get's resolve.

@perryibm
Copy link
Contributor

I think #441 is a duplicate. Basically, moving dockerSkeleton to open involves actually moving the directory over and making adjustments to the following:

  1. Build process (scripts and directory) which used to be in ant but now is in gradle
  2. One test that downloads the skeleton may need adjustment
  3. The CLI will have to be adjusted based on details from step 1

@csantanapr
Copy link
Member

@perryibm Sounds good I closed dup #441
I think the CLI change is minor and can be done in the current python CLI, if the work wants to get implemented currently.

I moved this issue from IceBox(Low) to the Backlog pipeline
Do you have intention to work on this? It's currently assigned to you.

@perryibm
Copy link
Contributor

I have some higher priority things to take care of. If you would like to take a crack at it, by all means take it.

@csantanapr
Copy link
Member

yep, I understand I will leave unassigned and free for anyone to take it, even you 😄
At this moment we are book also on higher priorities, but having unassigned it's good so it tells people it's ups for grabs

@daisy-ycguo
Copy link
Member

daisy-ycguo commented May 25, 2016

I have interests to fix issue. Please let me know if my understanding is correct.

The goal is to make dockerSkeleton tar ball available here: https:///blackbox-0.1.0.tar.gz, then command wsk sdk install docker can download dockerSkeleton and works.

The artifacts related with dockerSkeleton include:

catalog/demo/dockerSkeleton
tools/build.xml
tools/cli/publishArtifact.sh

My plan is to create a folder in OpenWhisk: tools/blackbox, and put all related artifacts there. It may take me several days to work on it, learning by doing. If you think it's OK, I can take this issue.

@csantanapr
Copy link
Member

@daisy-ycguo I'm OK if you want to work on this.
But we are trying move catalog content out to individual repositories.

I will create a new repository "openwhisk-sdk-docker"
Don't know if that will be the final name but we can use it for initial implementation

Think about what would be the other things todo

  • main README
    • how to run tests
    • how to contribute doc
  • unit tests
  • Travis and any othe continues integrations aspects
  • Updste this repo to depend on the new repo

@csantanapr
Copy link
Member

@daisy-ycguo I created the repo openwhisk/openwhisk-sdk-docker
Let's work together on this to populate with initial content

@csantanapr
Copy link
Member

@vmtyler This should be fix now, give it a try

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants