-
Notifications
You must be signed in to change notification settings - Fork 462
Fetch packages from package-storage #21
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
Conversation
💚 Build SucceededExpand to view the summary
Build stats
|
.gitignore
Outdated
@@ -3,3 +3,5 @@ | |||
.idea | |||
build | |||
public | |||
|
|||
/dev/packages/storage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we put it under build/packages/storage
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, this one doesn't need special cleaning. Fixed.
magefile.go
Outdated
tarGz = true | ||
publicDir = "./public" | ||
buildDir = "./build" | ||
storageDir = "./dev/packages/storage" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leftover, not needed anymore.
ey folks, trying to understand the workflow here. Why is integrations repo fetching packages from storage? |
Packages served by the package-registry are stored in the package-storage. There are some of them that are internal/required like: Currently there are no simple ways of attaching more packages to the registry. I can look into possibilities and modify the registry, but changes the scope of this task a bit. |
One reason are the required packages but there is also a very nice side effect here. As all the existing packages are pulled in, any testing of new packages together with Kibana is what it will look like in the end. Example: Someone is working on the nginx package 1.3.2. The current version shipped is 1.3.1. 1.3.1 will be pulled in from the package-storage and 1.3.2 will be on top of it. |
This point depends on outcome of the discussion here: #15 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change LGTM.
My understanding of this PR now:
- package-storage are checked out into
build
directory so only change related to this is in the magefile - The diff on the alpha packages are coming from (version-)changes since this repo was created (mostly).
Changes:
mage build
to include packages from package-storagemage build
multiple times (don't re-checkout the package-storage)remove(import recent alpha packages)dev/packages/alpha
as they are in the package-storage