Unexpected alias behaviour - two ranges

stephenw10

If you run filter_reload does it fully populate?

Mixed mode aliases have been a problem in the past. I've long recommended not mixing IPs and FQDNs but I had thought those issues were resolved. Looks like we have a regression.

SteveITS

The first time I tried this I had an error in my list for alias_512. I accidentally scrolled two extra rows, leaving this in the import copy/paste:

10.10.0.256
10.10.0.257

Obviously an error. Filterdns (DNS Resolver log) threw an error trying to resolve the "hostnames" since they are not IPs. The result was still the problem above, however, the results/numbers were slightly different and not off by two.

Deleting those and clicking Apply again reproduced the issue, hence my above post.

I'm not sure what that means but it seems odd that removing the two invalid IPs resulted in 1) several more (more than 2) additional IPs made it into the alias_512 table, and 2) the FQDN forum.netgate.com at the bottom of that list was resolved and its IPs also in that table. Even though 10.0.0.171-.255 and 10.10.0.1-.255 are not. Possibly an out of memory error and the "hostnames" take a bit more RAM than the last few IPs? I did try adding them back in again and the tables did not shrink as I expected.

Note also that as @Patch reported, the log shows the missing IPs being added:

Nov 7 01:34:37 	filterdns 	55828 	Adding Action: pf table: alias_all host: 10.10.0.253
Nov 7 01:34:37 	filterdns 	55828 	Adding Action: pf table: alias_all host: 10.10.0.254
Nov 7 01:34:37 	filterdns 	55828 	Adding Action: pf table: alias_all host: 10.10.0.255

SteveITS

@stephenw10 said in Unexpected alias behaviour - two ranges:

If you run filter_reload does it fully populate?

no.

Initializing
Creating aliases
Creating gateway group item...
Generating Limiter rules
Generating NAT rules
Creating 1:1 rules...
Creating outbound NAT rules
Creating automatic outbound rules
Setting up TFTP helper
Generating filter rules
Creating default rules
Pre-caching Default allow LAN to any rule...
Creating filter rule Default allow LAN to any rule ...
Creating filter rules Default allow LAN to any rule ...
Setting up pass/block rules
Setting up pass/block rules Default allow LAN to any rule
Creating rule Default allow LAN to any rule
Pre-caching Default allow LAN IPv6 to any rule...
Creating filter rule Default allow LAN IPv6 to any rule ...
Creating filter rules Default allow LAN IPv6 to any rule ...
Setting up pass/block rules
Setting up pass/block rules Default allow LAN IPv6 to any rule
Creating rule Default allow LAN IPv6 to any rule
Pre-caching Passed via EasyRule...
Creating filter rule Passed via EasyRule ...
Creating filter rules Passed via EasyRule ...
Setting up pass/block rules
Setting up pass/block rules Passed via EasyRule
Creating rule Passed via EasyRule
Creating IPsec rules...
Generating ALTQ queues
Loading filter rules
Setting up logging information
Setting up SCRUB information
Processing down interface states
Running plugins
Done

SteveITS

@stephenw10 said in Unexpected alias behaviour - two ranges:

not mixing IPs and FQDNs

If I remove forum.netgate.com from alias_512 then I get 618 records so I think it fully populates.

SteveITS

OK, weirder, I set the tunable for https://docs.netgate.com/pfsense/en/latest/troubleshooting/filterdns-thread-errors.html to 4096, just to see. I applied it, and did a Filter Reload.

alias_512 now contains only 12 records, scattered through the list plus the last 7+FQDN:

10.0.0.138 	
10.10.0.1 	
10.10.0.58 	
10.10.0.249 	
10.10.0.250 	
10.10.0.251 	
10.10.0.252 	
10.10.0.253 	
10.10.0.254 	
10.10.0.255 	
208.123.73.77 	
2610:160:11:11::6

However alias_all still contains 618 entries. Which makes me think it either was created successfully, or not updated at all.

Patch

@SteveITS

In my testing when editing alias, the old table entries are not deleted. But I agree sometimes not fully updated. This is probably why the bug results in latent failure, and the fault is revelled on pfsense restart when all aliases are fully built.
I agree it can be reproduced without restarting, it's just that I found it harder to make sense of the data.
I suspect deleting table data then filter reload would be equivalent.
Configuration save and reload is a belts & braces approach used as I didn't know what part of pfsense was locking up initially.
I get the same behaviour in pfsense 2.7.2 also. I looked as I wondered about a regression or new bug but could not confirm that.

@SteveITS said in Unexpected alias behaviour - two ranges:

the log shows the missing IPs being added

I think included IP's added once but missing IP are added twice.

To see this I

set the Resolver log to show 2000 entries
copy the log
paste text into a spreadsheet (such as LibreOffice calc) using space as a delimiter
sort by the description column (D)
scroll to Alias of interest & look for duplicates

SteveITS

https://redmine.pfsense.org/issues/9296 sounds similar, though the posters say killing filterdns and reloading filters fixes it. I did not test that but could, later today. And/or open a new redmine if desired. Note the last post says it was a problem in 2.7.2.

My issue I linked above actually sounds more like https://redmine.pfsense.org/issues/14734 (when the FQDN changes IPs the separately listed/duplicate IP is incorrectly removed from the table).

tinfoilmatt

@SteveITS said in Unexpected alias behaviour - two ranges:

The first time I tried this I had an error in my list for alias_512. I accidentally scrolled two extra rows, leaving this in the import copy/paste:

10.10.0.256
10.10.0.257

This 'bunkifies' your entire test—for all three aliases ("alias_50_1", "alias_50_2", an "alias_512") and the one nested alias ("alias_all").

Can you re-run this without any user error and see if you observe any 'consistency' of results? My guess is there will be enough difference (meaning, lack of consistency) in 'results' so as to mean nothing.

tinfoilmatt

@stephenw10 said in Unexpected alias behaviour - two ranges:

Mixed mode aliases

I like that. Has a nicer ring than 'kludge.'

SteveITS

@tinfoilmatt said in Unexpected alias behaviour - two ranges:

Can you re-run this without any user error

I guess that's fair though it would imply deleting the invalid entries still causes a problem? Or at least that doing so doesn't fix the problem. Onwards (read it all)...

If I delete all four aliases, apply, and re-import them, I do not see the error case today even after a filter reload or reboot. All four aliases are correct (618 total IPs).

I added "invalid" to alias_512 and applied, same.

I emptied all four tables, ran a filter reload, and all four remained empty.

I removed "invalid" and ran a filter reload, all tables remained empty.

I had to "killall filterdns" and filter reload, and after that the tables populated correctly.

next:
empty all tables
add "invalid" to alias_512, and apply
all tables remain empty
killall filterdns, and reload filter
all tables are populated correctly

...so, killing filterdns is suddenly required to get the tables to recreate at all. @stephenw10, does a filter reload actively empty the tables when it runs, or does it leave them and attempt to update them?

next:
I started over, imported the aliases with the extra two error lines, just like last night, and was unable to replicate my original observed case (incomplete aliases). Unclear why it is different today. I shut down the VM overnight, which seems irrelevant but did happen.

It seems there is definitely "something wrong" because the alias tables are either sometimes incomplete or empty, but now I'm confused also.

stephenw10

Yes I expect it to re-populate the tables based on the loaded ruleset.

It looks like there are at least two bugs still outstanding related to this. But as far as I know neither is a regression for 2.8.1/25.07.1.

@Patch you first saw this in 2.8.1? Is it possible it was happening in 2.7.2 and you just didn't notice?

Patch

@stephenw10 yes I first saw this in v2.81 and had not tripped it in v2.72

I then installed v2.72 in a VM using the current installer an explicit testing as per. https://forum.netgate.com/post/1229337 showed essentially the same behaviour.

The only real testing I had done after the error is triggered is to demonstrate creating a new trivial alias results in an alias table but it isn’t populated.

I avoided further testing as I had previously found repairing the system by further changing the alias definition was difficult. The system behaves as if something has crashed or locked up. My current experience is data entry errors are handled correctly by pfsense but the alias table filling error once triggered persist. Which initially miss lead me into blaming data entry error handling. Hence my very frequent restarts / configuration restore in testing

SteveITS

@stephenw10 said in Unexpected alias behaviour - two ranges:

Yes I expect it to re-populate the tables based on the loaded ruleset.

Yes, but the hair I'm splitting is whether the alias Apply is either 1) not updating the table as expected, or 2) not emptying the tables at the beginning of its run and thus presumably aborting very early in the process. Just thinking about the programming out loud, is all. Because if I manually empty them and they stay empty that implies the prior filter reload maybe didn't get to the point of emptying them.

I guess I didn't explain it well but it seems like:

I added "invalid" to alias_512 and applied, same.
...is possibly not a great test if I didn't "killall filterdns" and filter reload.

Seems like one possibility is filterdns gets stuck and thus the tables aren't updated. Which may be what @Patch is talking about when mentioning lockups.

tinfoilmatt

@SteveITS said in Unexpected alias behaviour - two ranges:

Onwards (read it all)...

Clearly I've been 'reading it all', Steve. Otherwise I wouldn't still be here. Does it concern you that I somehow keep picking the most relevant bits out of the noise to maintain my position here?

Your focus on the matter at-hand is showing with that comment (which I've of course taken the bait on and obliged you).

@SteveITS said in Unexpected alias behaviour - two ranges:

I had to "killall filterdns" and filter reload, and after that the tables populated correctly.

I had a feeling...

tinfoilmatt

@stephenw10 said in Unexpected alias behaviour - two ranges:

It looks like there are at least two bugs still outstanding related to this.

Redmine links?

tinfoilmatt

@SteveITS said in Unexpected alias behaviour - two ranges:

so, killing filterdns is suddenly required to get the tables to recreate at all.

Or you could just, like—not introduce user error and it probably wouldn't be necessary.

SteveITS

@tinfoilmatt said in Unexpected alias behaviour - two ranges:

Clearly I've been 'reading it all',

That wasn't directed at you, I just meant to read my whole post, there, since the behaviors changed.

@tinfoilmatt said in Unexpected alias behaviour - two ranges:

I had a feeling...

That wasn't the case last night, they did update on Apply.

@tinfoilmatt said in Unexpected alias behaviour - two ranges:

Or you could just, like—not introduce user error and it probably wouldn't be necessary.

What was the error you allege in today's post? AFAIK if I empty a table and filter reload, pfSense is supposed to populate the table.

tinfoilmatt

@SteveITS said in Unexpected alias behaviour - two ranges:

I just meant to read my whole post

I would hope anybody participating here and on the entire forum—nay, the entire Internet—thoroughly reads and considers in earnest any communtication directed at them by a fellow human being.

But back to topic at-hand, anything you did today is preempted by the fact that you didn't start with...

[in Unexpected alias behaviour - two ranges:]

I created a VM with 2.8.1.
I used easyrule to allow access on WAN.
I bypassed the GUI setup wizard.

...like you did yesterday. (Some people refer to this methodology colloquially as 'blowing everything out and starting over.') In other words you didn't even consistently recreate your own test.

I could break any system with some formulation of rm or system-specific equivalent. What does that tell anyone?

stephenw10

@tinfoilmatt said in Unexpected alias behaviour - two ranges:

I would hope anybody participating here and on the entire forum—nay, the entire Internet—thoroughly reads and considers in earnest any communtication directed at them by a fellow human being.

tinfoilmatt

@stephenw10 Hey, at least you're getting paid to placate this behavior. I'm only here to have fun!