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

RHDEVDOCS-3614 ODC Release Notes, New features for 4.10 #42818

Merged
merged 1 commit into from Mar 7, 2022
Merged

RHDEVDOCS-3614 ODC Release Notes, New features for 4.10 #42818

merged 1 commit into from Mar 7, 2022

Conversation

rishumehra
Copy link
Contributor

@rishumehra rishumehra commented Mar 4, 2022 •
edited

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 4, 2022
@netlify
Copy link

netlify bot commented Mar 4, 2022 •
edited

✔️ Deploy Preview for osdocs ready!

🔨 Explore the source changes: 3023647

🔍 Inspect the deploy log: https://app.netlify.com/sites/osdocs/deploys/62261dd987ac8d0008e8e948

😎 Browse the preview: https://deploy-preview-42818--osdocs.netlify.app

rolfedh
rolfedh reviewed Mar 4, 2022
View reviewed changes
Copy link
Contributor

@rolfedh rolfedh left a comment

Choose a reason for hiding this comment

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

@rishumehra I hope these suggestions are helpful.

rishumehra
rishumehra commented Mar 4, 2022
View reviewed changes
Copy link
Contributor Author

@rishumehra rishumehra left a comment

Choose a reason for hiding this comment

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

Thanks, @rolfedh @jerolimov for your reviews. I'll commit new updates soon.

release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
@rishumehra rishumehra closed this Mar 4, 2022
@rishumehra rishumehra reopened this Mar 4, 2022
jerolimov
jerolimov reviewed Mar 4, 2022
View reviewed changes
Copy link
Member

@jerolimov jerolimov left a comment

Choose a reason for hiding this comment

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

I don't verify if this contains all epics. But technically this lgtm 👍

/lgtm

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. and removed lgtm Indicates that a PR is ready to be merged. labels Mar 4, 2022
@jerolimov
Copy link
Member

jerolimov commented Mar 4, 2022 •
edited

Hey, I compared this now with our 13 epics we have delivered with 4.10. :)

First of all, I think this epics are already addressed well 👍

  • ODC-5947 Support Service Binding in Topology view
  • ODC-5506 Devfile catalog integration
  • ODC-5713 Namespaced Helm Chart Registries
  • ODC-6296 Enhance Export App UX experience

But maybe we want to mention some of these epics here. I added on the second level the feature I would mention. But if you want or don't want pick one up, and the option to aggregate them are obviously up to you. Maybe we don't want mention everything...? 😄

  • ODC-6266 Improve DevExp for front end developers (update March 7th: not included in Release Notes since this is an update)
    • There is a new option to override the npm start command when import an application from Git with s Node.js Builder Image.
    • We simplified the access to the target port field when importing an application.
    • After importing there is a new toast notification that shows the exported route URL with an option to copy it also.
    • The devfile catalog shows devfiles from the community devfile registry
  • ODC-6303 Dev Console Usability Improvements (update March 7th: has no-docs label)
    • The add page has now the option to open the same Quick Add/Search modal similar to the topology
    • There are new user preferences to define the default security options for new Deployments
  • ODC-6359 ODC/Serverless Usability Improvements (update March 7th: documented in Serverless docs)
    • Improved visualization of Kamelet sink resource in topology
  • ODC-6372 Add flows should allow users to pick from a list of pipelines (update March 7th: not included in Release Notes since this is an update)
    • Custom pipelines are now also shown on the import page and could be used when import an application from source or container image.
  • ODC-6361 ODC/Topology Usability Improvements (update March 7th: has no-docs label)
    • Users can now resize the sidebar in topology based on their needs.
    • The latest selection was also kept open when switching between topology and other pages.
    • The topology 'Open URL' decorator on Deployments could now be customized with some annotations.

There are many topology-related improvements... Maybe it's enough to aggregate them?

I also agree that we don't have to add these two epics (IMHO!), as well as the automation epic. I can see that the dynamic plugin epic is already mentioned below your list.

  • ODC-6375 Guide user to add webhook for pipeline templates
  • ODC-6377 ODC/Pipelines TektonHub Improvements

@jerolimov
Copy link
Member

Thanks Rishu for checking my list above and the clarification that these things should not be included in the release notes. Added some individual updates above why.

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 7, 2022
@Preeticp
Copy link
Contributor

Preeticp commented Mar 7, 2022 •
edited

I recommend adding ODC-6375 Guide user to add webhook for pipeline templates and
ODC-6377 ODC/Pipelines TektonHub Improvements, as these are important features from the end user POV for Pipelines.
Maybe add three nested-bullets instead of this one: * With this update, you can see when the pipeline has been created but not started by looking at the status of the pod in the Topology view.
Something like -
The creating pipelines workflow has now been enhanced:

  • You can see when the pipeline has been created but not started by looking at the status of the pod in the Topology view.
  • webhook related info
  • Hub integration

@rishumehra
Copy link
Contributor Author

I recommend adding ODC-6375 Guide user to add webhook for pipeline templates and ODC-6377 ODC/Pipelines TektonHub Improvements, as these are important features from the end user POV for Pipelines. Maybe add three nested-bullets instead of this one: * With this update, you can see when the pipeline has been created but not started by looking at the status of the pod in the Topology view. Something like - The creating pipelines workflow has now been enhanced:

  • You can see when the pipeline has been created but not started by looking at the status of the pod in the Topology view.
  • webhook related info
  • Hub integration

@Preeticp Thanks for the suggestion I'll add it in further edits.

@openshift-ci
Copy link

openshift-ci bot commented Mar 7, 2022

New changes are detected. LGTM label has been removed.

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Mar 7, 2022
Preeticp
Preeticp reviewed Mar 7, 2022
View reviewed changes
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated
* With this update, creating pipelines workflow has now been enhanced:
** You can see when the pipeline has been created but not started by looking at the status of the pod in the *Topology* view.
** You can create an application using the *Import from Git* workflow, along with a template-generated pipeline by selecting the *Add pipeline* option. The webhook resources and URL are visible in the side panel of the selected resources.
** You can opt out of the default Tekton Hub integration by setting the parameter `enable-devconsole-integration` to `false` in the `TektonConfig` custom resource. When you opt out, only the platform compatible cluster tasks for your namespace are listed in the *Pipeline builder* -> *Add tasks* view.
Copy link
Contributor

Choose a reason for hiding this comment

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

@karthikjeeyar can you please verify if this is technically correct?

rishumehra
rishumehra commented Mar 7, 2022
View reviewed changes
Copy link
Contributor Author

@rishumehra rishumehra left a comment

Choose a reason for hiding this comment

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

@karthikjeeyar @Preeticp thanks for your reviews and comment. I'll try to rephrase the content as per the suggestions.

release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
release_notes/ocp-4-10-release-notes.adoc Outdated Show resolved Hide resolved
karthikjeeyar
karthikjeeyar approved these changes Mar 7, 2022
View reviewed changes
Copy link

@karthikjeeyar karthikjeeyar left a comment

Choose a reason for hiding this comment

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

LGTM

@Preeticp Preeticp added branch/enterprise-4.10 dev-tools Label for all Odo/Pipelines/Helm/Developer Console/Perspective PRs lgtm Indicates that a PR is ready to be merged. peer-review-done Signifies that the peer review team has reviewed this PR labels Mar 7, 2022
@Preeticp Preeticp merged commit 3cfa39c into openshift:enterprise-4.10 Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jerolimov jerolimov jerolimov left review comments

@Preeticp Preeticp Preeticp left review comments

@rolfedh rolfedh rolfedh approved these changes

@karthikjeeyar karthikjeeyar karthikjeeyar approved these changes

Assignees

@jerolimov jerolimov

Labels
branch/enterprise-4.10 dev-tools Label for all Odo/Pipelines/Helm/Developer Console/Perspective PRs lgtm Indicates that a PR is ready to be merged. peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@rishumehra @jerolimov @Preeticp @rolfedh @karthikjeeyar