<p style="white-space: pre-wrap; word-wrap: break-word;">regarding usefulness of this patch: I thought you really had a specific scenario in mind. Otherwise, if you want to uspport both v4 and v6 addresses for default APN, simply set default-apn to an APN supporting both v4 and v6...</p><p style="white-space: pre-wrap; word-wrap: break-word;">I agree with you the only real use for the feature you are adding is in the event you really want to have default working configurations for PDP contexts requesting v4, v6 or v4v6. If you think that's useful we can merge it, otherwise I see no point.</p><p>Patch set 5:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941/5/ggsn/ggsn.c">File ggsn/ggsn.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941/5/ggsn/ggsn.c@475">Patch Set #5, Line 475:</a> <code style="font-family:monospace,monospace">                              apn = ggsn->cfg.default_apn;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Can we have this renamed to default_apn_v4?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941/5/ggsn/ggsn_vty.c">File ggsn/ggsn_vty.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941/5/ggsn/ggsn_vty.c@274">Patch Set #5, Line 274:</a> <code style="font-family:monospace,monospace">       "Set a default-APN to be used if no other APN v4 matches\n"</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">You are missing a line here I think (for v4 word).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Btw, better use "default-apn (v4|v6|v4v6) NAME" and implement the 3 of them in one command. Same for the "no" commands.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941/5/ggsn/ggsn_vty.c@306">Patch Set #5, Line 306:</a> <code style="font-family:monospace,monospace">    "Set a default-APN to be used if no other APN v6 matches\n"</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">same, missing line (on all of them)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941">change 17941</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-ggsn/+/17941"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-ggsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I03fcf8a1532bd9988ea99a6afd3dc325174ce9d6 </div>
<div style="display:none"> Gerrit-Change-Number: 17941 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: keith <keith@rhizomatica.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 26 Apr 2020 15:39:28 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>