OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. #2520

ankedia · 2021-08-31T18:20:56Z

The documentation for console and WLST access using kubectl port-forward and changes to handle multiple custom admin channel use-case. Added unit tests for the admin port enabled and multiple custom admin channel use-case.

…ss using kubectl port-forward.

tbarnes-us · 2021-08-31T19:03:01Z

General comments:

The "access a domain" remote console chapter should be updated to mention the port-forward option.
The "access a domain" WLST chapter is in general need of update.
I have a few edits in mind for the new chapter, but don't have time right now. May I have the 'token' so that I can make them directly by say, Thursday EOD?

[ANIL]

I'm not familiar with the remote console, will go through that chapter to see where port-forward option can be mentioned.
Ok.
Sure, go ahead. In case I need to make any changes based on other review comments or if I remember something, I will keep them local for now and merge them with your changes after you're done editing. Thanks.

jshum2479 · 2021-08-31T20:35:49Z

documentation/staging/content/userguide/managing-domains/accessing-the-domain/port-forward.md

+```
+Forwarding from 127.0.0.1:32015 -> 7001
+```
+


Should there be a cleanup notes for ending the forwarding session?

Added a Terminating the port-forward session section in 0d1a9a1 .

…sion and minor changes.

…sole and WLST chapters.

…tails and a warning message.

tbarnes-us · 2021-09-04T01:40:39Z

@jshum2479 and @rjeberhard, I've pushed a number of doc updates on top of Anil's updates.

If you two are fine with the doc, then it is ready for @rosemarymarano to make a pass.

Ryan, I'd especially like you to review all of my version number references (search for 4.0 and 3.3).

rjeberhard · 2021-09-07T17:40:39Z

documentation/staging/content/security/domain-security/weblogic-channels.md

+
+#### WebLogic HTTP channels
+
+When providing remote access to HTTP using a load balancer or similar,


"or similar" is confusing here. I know that we mean something like an Ingress gateway, but it's not obvious here. It might be worth spelling it out a bit more such as "load balancer, Ingress gateway, or similar technology,"

rjeberhard · 2021-09-07T17:42:12Z

documentation/staging/content/security/domain-security/weblogic-channels.md

+is preferred over a default port because a default port supports
+multiple protocols.
+
+Do not enable tunneling on an HTTP channel unless you specifically


One of the new security validations complains if tunneling is simply enabled. Given that, we probably should more clearly recommend against using tunneling unless the port is more protected on the network.

rjeberhard · 2021-09-07T17:52:57Z

documentation/staging/content/userguide/managing-domains/accessing-the-domain/port-forward.md

+
+### Overview
+
+Beginning with WebLogic Kubernetes Operator version 3.3.2,


This looks fine to me, but we should ask Rosemary to help us wordsmith this. There probably is some simpler way to express this. :)

@rosemarymarano Once the ball is in your court, please see ^^^.

@tbarnes-us @rjeberhard Sure. "...help us wordsmith this. There probably is some simpler way to express this.
One question: to what are you referring as "this," the Overview section or something else?

I thought that the wording was fine.

…netes-operator into owls_91862

ankedia · 2021-09-08T18:26:53Z

@rjeberhard, @jshum2479, and @lennyphan - Can you please re-review my changes to handle multiple custom admin channels in 82672c1 ? Thanks in advance.

…netes-operator into owls_91862

lennyphan

LGTM. Any unit tests for adding multiple internal admin channels?

[ANIL] I have added unit tests for multiple internal admin channels and when admin port is enabled in 0b9d8b8 . Please let me know if I missed something.

…els and minor change in logic to determine admin port enabled for MII domain.

….6+.

…port-forward and code changes to handle multiple custom admin channels. (#2520) * Support kubectl port-forward pattern Co-authored-by: Tom Barnes <tom.barnes@oracle.com> Co-authored-by: Rosemary Marano <rosemary.marano@oracle.com>

…nChannelPortForwardingEnabled to false. (#2532) * Owls 91273 - Create network access points for kubectl port-forward and console access (#2507) * Create network access points for kubectl port-forward and console access * OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. (#2520) * Support kubectl port-forward pattern Co-authored-by: Tom Barnes <tom.barnes@oracle.com> Co-authored-by: Rosemary Marano <rosemary.marano@oracle.com> * Change adminChannelPortForwardingEnabled default to false. * Change the default ADMIN_CHANNEL_PORT_FORWARDING_ENABLED env variable value to false. * Enabled port forwarding for unit tests. Co-authored-by: Tom Barnes <tom.barnes@oracle.com> Co-authored-by: Rosemary Marano <rosemary.marano@oracle.com>

OWLS-91862 - Initial draft of documentation for console and WLST acce…

8bbfc26

…ss using kubectl port-forward.

ankedia requested review from jshum2479, rjeberhard and tbarnes-us August 31, 2021 18:20

Minor changes.

73df007

jshum2479 reviewed Aug 31, 2021

View reviewed changes

ankedia added 3 commits August 31, 2021 21:58

PR review comment - added section on terminating the port-forward ses…

0d1a9a1

…sion and minor changes.

PR review comments to add kubectl port-forward option to remote con…

7fd7a78

…sole and WLST chapters.

Chnages to WLST chapter to remove exposeAdminT3Channel, add more de…

e984e54

…tails and a warning message.

ankedia marked this pull request as ready for review September 2, 2021 17:17

ankedia changed the title ~~OWLS-91862 - Draft documentation for console and WLST access using kubectl port-forward.~~ OWLS-91862 - Documentation for console and WLST access using kubectl port-forward. Sep 2, 2021

doc updates

9e4241b

Tom Barnes added 2 commits September 7, 2021 14:59

minor doc edits

47a9f18

some more minor doc edits

bac802c

rjeberhard reviewed Sep 7, 2021

View reviewed changes

doc updates based on review comments

20b82c2

tbarnes-us approved these changes Sep 7, 2021

View reviewed changes

rjeberhard approved these changes Sep 7, 2021

View reviewed changes

rosemarymarano and others added 4 commits September 8, 2021 10:51

beginning edits

cef4c90

Merge branch 'owls_91862' of https://github.com/oracle/weblogic-kuber…

c8f8551

…netes-operator into owls_91862

Merge remote-tracking branch 'origin/main' into owls_91862

4053478

Changes to handle multiple custom admin channel configuration use-case.

82672c1

rosemarymarano added 2 commits September 8, 2021 14:38

edits

6d4a100

Merge branch 'owls_91862' of https://github.com/oracle/weblogic-kuber…

8cf1459

…netes-operator into owls_91862

lennyphan approved these changes Sep 8, 2021

View reviewed changes

ankedia added 3 commits September 8, 2021 22:51

PR review comments - added unit tests for multiple custom admin chann…

0b9d8b8

…els and minor change in logic to determine admin port enabled for MII domain.

Delete CodeQL analysis GitHub action as discussed with Ryan.

40447d6

Restore CodeQL analysis and fix an issue with python versions above 3…

5453baf

….6+.

ankedia changed the title ~~OWLS-91862 - Documentation for console and WLST access using kubectl port-forward.~~ OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. Sep 9, 2021

Change to revert the fix for an issue caused by older WDT version.

85d20d6

rosemarymarano approved these changes Sep 9, 2021

View reviewed changes

rjeberhard merged commit 368784f into main Sep 9, 2021

ankedia deleted the owls_91862 branch September 10, 2021 13:17

ankedia mentioned this pull request Sep 13, 2021

Merge port forward changes to release/3.3 and change default for adminChannelPortForwardingEnabled to false. #2532

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. #2520

OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. #2520

Uh oh!

ankedia commented Aug 31, 2021 •

edited

Loading

Uh oh!

tbarnes-us commented Aug 31, 2021 •

edited by ankedia

Loading

Uh oh!

jshum2479 Aug 31, 2021

Uh oh!

ankedia Aug 31, 2021

Uh oh!

tbarnes-us commented Sep 4, 2021

Uh oh!

rjeberhard Sep 7, 2021

Uh oh!

tbarnes-us Sep 7, 2021

Uh oh!

rjeberhard Sep 7, 2021

Uh oh!

tbarnes-us Sep 7, 2021

Uh oh!

rjeberhard Sep 7, 2021

Uh oh!

tbarnes-us Sep 7, 2021

Uh oh!

rosemarymarano Sep 7, 2021

Uh oh!

rosemarymarano Sep 9, 2021

Uh oh!

ankedia commented Sep 8, 2021

Uh oh!

lennyphan left a comment •

edited by ankedia

Loading

Uh oh!

Uh oh!


		#### WebLogic HTTP channels

		When providing remote access to HTTP using a load balancer or similar,


		### Overview

		Beginning with WebLogic Kubernetes Operator version 3.3.2,

OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. #2520

OWLS-91862 - Documentation for console and WLST access using kubectl port-forward and code changes to handle multiple custom admin channels. #2520

Uh oh!

Conversation

ankedia commented Aug 31, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

tbarnes-us commented Aug 31, 2021 • edited by ankedia Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tbarnes-us commented Sep 4, 2021

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ankedia commented Sep 8, 2021

Uh oh!

lennyphan left a comment • edited by ankedia Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ankedia commented Aug 31, 2021 •

edited

Loading

tbarnes-us commented Aug 31, 2021 •

edited by ankedia

Loading

lennyphan left a comment •

edited by ankedia

Loading