Skip to content

Commit

Permalink
Review feedback
Browse files Browse the repository at this point in the history
- Addrobj name typo
- Cleanup zone setup manifest
  • Loading branch information
bnaecker committed Jul 25, 2024
1 parent f7d80f1 commit 6c93f59
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion illumos-utils/src/addrobj.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ pub const IPV6_STATIC_ADDROBJ_NAME: &str = "omicron6";

/// The name provided to all static IPv4 addresses, usually for public OPTE
/// interfaces.
pub const IPV4_STATIC_ADDROBJ_NAME: &str = "omicron6";
pub const IPV4_STATIC_ADDROBJ_NAME: &str = "omicron4";

/// The name provided to DHCP-configured addresses, of either family.
pub const DHCP_ADDROBJ_NAME: &str = "omicron";
Expand Down
4 changes: 2 additions & 2 deletions smf/zone-network-setup/manifest.xml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
<create_default_instance enabled='true' />

<!-- Run after the zone's networking stack is up. -->
<dependency name='physical' grouping='require_all' restart_on='none'
<dependency name='network' grouping='require_all' restart_on='none'
type='service'>
<service_fmri value='svc:/milestone/network:default' />
</dependency>
Expand All @@ -16,7 +16,7 @@
have been set by the sled-agent, which happens after the
`manifest-import` service is running.
-->
<dependency name='zone-network-setup-manifest-import' type='service' grouping='require_all' restart_on='none'>
<dependency name='manifest-import' type='service' grouping='require_all' restart_on='none'>
<service_fmri value='svc:/system/manifest-import:default' />
</dependency>

Expand Down

0 comments on commit 6c93f59

Please sign in to comment.