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.

Sign up for GitHub

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

Jump to bottom

Changelog for 0.9 release #578

Merged
merged 4 commits into from
Oct 30, 2015
Merged

Changelog for 0.9 release #578

merged 4 commits into from
Oct 30, 2015

Conversation

tomwilkie
Copy link
Contributor

No description provided.

@tomwilkie tomwilkie changed the title Changelog for 0.9 release [WIP] Changelog for 0.9 release Oct 20, 2015
peterbourgon
peterbourgon reviewed Oct 22, 2015
View reviewed changes
CHANGELOG.md
[#441](https://github.com/weaveworks/scope/pull/441)
- Add containers-by-hostname view
[#545](https://github.com/weaveworks/scope/pull/545)
- Make ```scope launch``` work from remote hosts, with an appropriately defined DOCKER_HOST

This comment was marked as abuse.

Sign in to view

@tomwilkie tomwilkie changed the title [WIP] Changelog for 0.9 release Changelog for 0.9 release Oct 22, 2015
@2opremio
Copy link
Contributor

I have tested this branch with the ECS demo and I am missing edges between the http server and the data producers:

screen shot 2015-10-26 at 2 11 34 pm

here's the report: https://gist.github.com/2opremio/455b0c294e603748cd5e

And here are the demo machines in case @tomwilkie wants to have a look at them:

  http://ec2-52-30-203-246.eu-west-1.compute.amazonaws.com
  http://ec2-52-30-243-163.eu-west-1.compute.amazonaws.com
  http://ec2-52-30-1-231.eu-west-1.compute.amazonaws.com

Also, the CLI now prints an error when checking the docker version: #588

@tomwilkie
Copy link
Contributor Author

Thanks Fons! If you tested with 1.2, this regression is caused by that: weaveworks/weave#1577

I suggest you disable FDP and test again.

@2opremio
Copy link
Contributor

Yep, I was running FDP. As for the second (admittedly minor) bug, I have a PR for this branch: #591

@tomwilkie tomwilkie force-pushed the 0.9 branch 3 times, most recently from 81ab028 to 63ae144 Compare October 28, 2015 15:28
paulbellamy
paulbellamy reviewed Oct 28, 2015
View reviewed changes
CHANGELOG.md
- Add support for loading the Scope UI from https endpoings
[#572](https://github.com/weaveworks/scope/pull/572)

Big fixes:

This comment was marked as abuse.

Sign in to view

@paulbellamy
Copy link
Contributor

I can't seem to replicate the fix for #493, @tomwilkie mind showing me what I'm doing wrong?

tomwilkie and others added 4 commits October 29, 2015 11:35
@tomwilkie
Don't keep stacking filters onto renderers; instead, ensure we return…
c6604a6 
… a fresh renderer each time.
@peterbourgon
Merge pull request #603 from weaveworks/602-app-cpu
3c13198 
Don't keep stacking filters onto renderers, close idle http connections when we close a http publisher.
@peterbourgon
Merge pull request #605 from weaveworks/604-close-idle-http
7dad389 
Close idle http connections when we stop a Http Publisher.
@tomwilkie
docker run -ti is problematic for automation.
e0cea0e
@peterbourgon
Copy link
Contributor

While I understand there are some remaining issues with conntrack, my issues are resolved; LGTM.

@tomwilkie tomwilkie merged commit e0cea0e into master Oct 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tomwilkie @2opremio @paulbellamy @peterbourgon