Skip to content

Refine deployment from Jenkinsfile, add timestamps for access in Folo tracked entries#1399

Merged
jdcasey merged 3 commits intoCommonjava:masterfrom
jdcasey:master
Oct 30, 2019
Merged

Refine deployment from Jenkinsfile, add timestamps for access in Folo tracked entries#1399
jdcasey merged 3 commits intoCommonjava:masterfrom
jdcasey:master

Conversation

@jdcasey
Copy link
Copy Markdown
Member

@jdcasey jdcasey commented Oct 24, 2019

We're missing information about how many times a given file is accessed during a build, and what the time distribution looks like between file accesses. This isn't needed for normal production use cases, but having it will give us a much better ability to simulate build users without actually running builds. With the timing information, we can sort the download / upload lists into a single "script" of artifact accesses and sizes, the replay it in a simulated environment.

We'll need more tracking information on the metadata files that get used during build alignment, but this is a step in the right direction.

Also, in the Jenkinsfile I'm making artifact deployment conditional on having an image build configuration available. This will limit CI scenarios that build for non-Commonjava forks of the repository, since those user-based forks will not have corresponding image build configs in our CI system. We don't want these non-Commonjava artifacts to be deployed, since that would make it much harder to figure out where a given snapshot came from.

@jdcasey jdcasey requested review from ruhan1 and sswguo October 24, 2019 22:25
Copy link
Copy Markdown
Contributor

@ruhan1 ruhan1 left a comment

Choose a reason for hiding this comment

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

LGTM

… so we can use Externalizable versioning for deser
@jdcasey jdcasey merged commit 8664bf9 into Commonjava:master Oct 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants