Skip to content

Anaconda network page - #23341

Draft
Venefilyn wants to merge 1 commit into
cockpit-project:mainfrom
Venefilyn:anaconda/networking-config
Draft

Anaconda network page#23341
Venefilyn wants to merge 1 commit into
cockpit-project:mainfrom
Venefilyn:anaconda/networking-config

Conversation

@Venefilyn

@Venefilyn Venefilyn commented Jun 1, 2026

Copy link
Copy Markdown
Member

Anaconda network page to simplify layout for Anaconda users.

Currently a WIP

image image


const managedWired: ListingTableRowProps[] = [];
const managedWireless: ListingTableRowProps[] = [];
let hasDetails = false;
});

// TODO: Actions: turn on off action and edit (wired) or join wifi (wireless)
const actions = privileged && (
}
]}
isDisabled={!privileged}
/>
const unmanaged = [];
const plot_ifaces = [];
let hasDetails = false;
const anaconda_mode = JSON.parse(window.sessionStorage.getItem("cockpit_anaconda"));
@Venefilyn
Venefilyn force-pushed the anaconda/networking-config branch from f3e8c6d to 1a1f051 Compare June 1, 2026 15:45

@cockpituous cockpituous left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There are more than 10 code coverage comments, see the full report here.

Signed-off-by: Freya Gustavsson <freya@venefilyn.se>
@Venefilyn
Venefilyn force-pushed the anaconda/networking-config branch from 1a1f051 to f84abff Compare August 4, 2026 15:13

@cockpituous cockpituous left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There are more than 10 code coverage comments, see the full report here.

Comment on lines +40 to +41
export const AnacondaNetworkPage = ({ privileged, operationInProgress, usage_monitor, interfaces }: AnacondaNetworkPageProps) => {
const [active, setActive] = useState<AnacondaActiveNetwork>();

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 2 added lines are not executed by any test. Details

Comment on lines +44 to +46
const managedWired: ListingTableRowProps[] = [];
const managedWireless: ListingTableRowProps[] = [];
let hasDetails = false;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 3 added lines are not executed by any test. Details

const managedWireless: ListingTableRowProps[] = [];
let hasDetails = false;

interfaces.forEach(iface => {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This added line is not executed by any test. Details

Comment on lines +50 to +51
if (is_loopback(iface))
return;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 2 added lines are not executed by any test. Details

Comment on lines +54 to +55
else if (has_group(iface))
return;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 2 added lines are not executed by any test. Details

else if (has_group(iface))
return;

const dev = iface.Device;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This added line is not executed by any test. Details

Comment on lines +62 to +63
const activeConnection = render_active_connection(dev, false, true);
const isWireless = is_wireless(iface);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 2 added lines are not executed by any test. Details

Comment on lines +65 to +67
let connectionStatus;
if (activeConnection) {
connectionStatus = _("Connected")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 3 added lines are not executed by any test. Details

if (activeConnection) {
connectionStatus = _("Connected")
} else {
connectionStatus = _("Disconnected")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This added line is not executed by any test. Details

Comment on lines +72 to +77
const row = (
<SimpleListItem key={iface.name} onClick={() => {setActive({isWireless, iface})}}>
<Flex
direction={{ default: 'row' }}
justifyContent={{ default: 'justifyContentSpaceBetween' }}
flexWrap={{ default: 'nowrap' }}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These 6 added lines are not executed by any test. Details

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