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.

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

1.1 Release Notes #3558

Merged
merged 1 commit into from
Mar 14, 2019
Merged

1.1 Release Notes #3558

merged 1 commit into from
Mar 14, 2019

Conversation

geeknoid
Copy link
Contributor

@geeknoid geeknoid commented Mar 7, 2019

Here's the initial translation of the Wiki release notes into the official format.

LEADS: Please review your areas and supply any missing links and missing content. You might realize that you need to write or update a doc page to support these notes.

@geeknoid
Copy link
Contributor Author

geeknoid commented Mar 7, 2019

FYI @oaktowner


## Policies and telemetry

- **Policy Checks Off By Default**. Policy checks are now turned off by default which improves performance for most customer scenarios.
Copy link
Contributor

Choose a reason for hiding this comment

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

i think we probably also need to add notes about disabling access logs by default too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, please suggest the right text.

- Enabled full disablement of tracing for Mixer-backed services TBD: REPHRASE
- Added policy decision-aware tracing. TBD: LINK

- **Default TCP Metrics**. There are now default metrics for tracking TCP connections.
Copy link
Contributor

Choose a reason for hiding this comment

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

should we expand this to include work to add labels like response_flags to the default metrics (which enables better monitoring of Fault Injections and Circuit Breaking) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Please recommend the right text.

- **Kiali**. [Kiali](https://www.kiali.io) replaces the [Service Graph addon](https://github.com/istio/istio/issues/9066) and provides
a richer visualization experience. See the [Kiali Task](/docs/tasks/telemetry/kiali/) for more details.

- **Reduced Overhead**. There have been several performance and scale improvements including:
Copy link
Contributor

Choose a reason for hiding this comment

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

worth calling out other perf improvements? (work with some of our default adapters, including fluentd, work with our dependencies including trace libraries, prometheus, etc.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, definitely worth it. Could you or @mandarjog suggest the right text please?

Copy link
Contributor

@rcaballeromx rcaballeromx left a comment

Choose a reason for hiding this comment

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

Added comments with the links I knew of and making every entry consistent with the others.

content/about/notes/1.1/index.md Outdated Show resolved Hide resolved

- **Allow All Outbound Traffic**. Allow all outbound traffic by default now. To enforce no outbound traffic at mesh wide, configure ```global.outboundTrafficPolicy.mode``` to ```REGISTRY_ONLY```

- **Egress Gateway Disabled By Default**.- Egress gateway is disabled by default now. To enable it, configure gateways.istio-egressgateway.enabled to ```true```
Copy link
Contributor

Choose a reason for hiding this comment

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

Where and how do readers enable it and configure gateways.istio-egressgateway.enabled to true? Please link to the task detailing how to perform this changes instead.

content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
@istio-testing
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geeknoid

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@geeknoid geeknoid requested a review from a team March 8, 2019 15:00
@geeknoid
Copy link
Contributor Author

geeknoid commented Mar 8, 2019

@frankbu @rshriram Networking section needs a lot of TLC...

@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of all the commit author(s), set the cla label to yes (if enabled on your project), and then merge this pull request when appropriate.

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. and removed cla: yes labels Mar 10, 2019
@rshriram
Copy link
Member

I have made changes to the traffic management section.

@googlebot
Copy link

CLAs look good, thanks!

ℹ️ Googlers: Go here for more info.

@googlebot googlebot added cla: yes and removed cla: no Set by the Google CLA bot to indicate the author of a PR has not signed the Google CLA. labels Mar 10, 2019
content/about/notes/1.1/index.md Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
TBD: need full content & links
- Egress gateway is disabled by default
- Mixer policy is off by default
- Outbound traffic policy is set to ALLOW_ANY. Traffic to unknown ports will be forwarded as is.
Copy link
Member

Choose a reason for hiding this comment

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

helm API changes need to be highlighted here.

Copy link
Contributor

@rcaballeromx rcaballeromx left a comment

Choose a reason for hiding this comment

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

Added suggestions for consistency and readability.

content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
@geeknoid
Copy link
Contributor Author

Gang, this is a P0 release blocker. Can you spend a bit of time addressing these relatively minor issues in the release notes?

@linsun @sdake @frankbu Can you address the remaining Helm items?

@liminw I added a new suggested items in the security section with a TBD. Please tell me what that needs to be, or we can delete the entry.

@costinm @sdake Can you suggest the needed stuff for the multicluster section?

@douglas-reid @mandarjog Lots of tweaks needed in the policy and telemetry content.

content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Outdated Show resolved Hide resolved
content/about/notes/1.1/index.md Show resolved Hide resolved
content/about/notes/1.1/index.md Show resolved Hide resolved
@geeknoid
Copy link
Contributor Author

@douglas-reid It's OK to remove links to docs that don't exist yet, but let's open an issue to track adding the missing doc. An undocumented new feature is not really usable by our customers...

Thanks.

@geeknoid
Copy link
Contributor Author

Deploy preview for preliminary-istio failed.

Built with commit 2fa8a52

https://app.netlify.com/sites/preliminary-istio/deploys/5c89e070db9a760008465406

@geeknoid geeknoid merged commit e76ecb4 into istio:master Mar 14, 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.

None yet