Skip to content

Conversation

@nwang92
Copy link
Contributor

@nwang92 nwang92 commented Oct 21, 2019

This branch will fail until the bits have been published, but I'm updating the version and hash anyways because we'll want it to tie off the release.

@nwang92
Copy link
Contributor Author

nwang92 commented Oct 21, 2019

Hmm maybe this PR should go directly into master?

arctan5x
arctan5x previously approved these changes Oct 21, 2019
hendolim
hendolim previously approved these changes Oct 21, 2019
@arctan5x arctan5x dismissed stale reviews from hendolim and themself via a309b6e October 22, 2019 02:43
echo "current $1 image size = "$$CUR_SIZE ; \
echo "edge image size = "$$EDGE_SIZE ; \
if [[ $$CUR_SIZE -gt $$EDGE_SIZE*120/100 ]] ; then echo "current image size is 20% more than edge image" ; exit 1 ; fi
if [[ $$CUR_SIZE -gt $$EDGE_SIZE*140/100 ]] ; then echo "current image size is 40% more than edge image" ; exit 1 ; fi
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it the splunk binary that has gotten so big?

Copy link
Contributor

Choose a reason for hiding this comment

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

it was ad hoc change in order to accommodate the release ;)

Copy link
Contributor

@arctan5x arctan5x Oct 22, 2019

Choose a reason for hiding this comment

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

Also, Splunk 8.0.0 vs 7.3.2 saw about 30% increase for linux distro when unextraced.

@arctan5x arctan5x merged commit 4605583 into develop Oct 22, 2019
@arctan5x arctan5x deleted the up-splunk-version branch October 22, 2019 18:35
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.

3 participants