Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

Documentation: document image lifecycle and implementation. #2911

Merged
merged 1 commit into from
Aug 5, 2016

Conversation

sgotti
Copy link
Contributor

@sgotti sgotti commented Jul 12, 2016

Document the image lifecycle phases, their logic and explain the rkt implementations.

---------------------
```

Currently rkt implements the appc spec, in future additional specs like (OCI) may be implemented keeping the same logic but chaning the implementations.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/chaning/changing/

@lucab
Copy link
Member

lucab commented Jul 12, 2016

I've spotted some typos, but I guess @joshix may have additional inputs to reword some of these sentences. Great to have this information written down, the decoupling part at the bottom was not clear to me before.

@sgotti
Copy link
Contributor Author

sgotti commented Jul 12, 2016

@lucab Thanks. Obviously there are better way to write the sentences 😄

The real intent is to explain how and why it works in this way, just to gather additional thoughts, better ideas and to have a starting point for future work on implementing the oci image spec.

@s-urbaniak
Copy link
Contributor

@joshix ping :-)

@sgotti sgotti force-pushed the image_lifecycle_architecture branch from 15ea291 to 0e2664d Compare July 27, 2016 07:13
@lucab
Copy link
Member

lucab commented Aug 3, 2016

@joshix @Phillels @ryanj Can any of you please have a proof-reading pass on this?

@joshix
Copy link
Contributor

joshix commented Aug 3, 2016

ack, looking, sorry about the delay. Will review today.

@lucab
Copy link
Member

lucab commented Aug 3, 2016

@joshix thanks, no problem. I was just going through milestoned PRs as tomorrow is release day.


### Image lifecycle

rkt commands like prepare and run, as a first step, needs to retrieve all the required images provided in the command line and prepare the stage2 directories with the application contents.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/needs/need/
s/required images provided/images requested/

@lucab lucab added this to the v1.13.0 milestone Aug 4, 2016
@lucab lucab removed this from the v1.12.0 milestone Aug 4, 2016
Document the image lifecycle phases, their logic and explain the rkt implementations.
@sgotti sgotti force-pushed the image_lifecycle_architecture branch from 0e2664d to 864f661 Compare August 5, 2016 16:02
@sgotti
Copy link
Contributor Author

sgotti commented Aug 5, 2016

@joshix @tmrts Thanks for the tough work decrypting my sentences! Updated.

@lucab
Copy link
Member

lucab commented Aug 5, 2016

LGTM

@lucab lucab merged commit 47b42f0 into rkt:master Aug 5, 2016
@lucab lucab unassigned joshix Apr 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants