Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5.15 Release #1332

Merged
merged 33 commits into from
Apr 20, 2022
Merged

5.15 Release #1332

merged 33 commits into from
Apr 20, 2022

Conversation

PromoFaux
Copy link
Member

Release PR

PromoFaux and others added 30 commits February 16, 2022 15:49
sync: master to development
Fix negative "others" in Upstream Servers pie chart
Signed-off-by: DL6ER <dl6er@dl6er.de>
Signed-off-by: DL6ER <dl6er@dl6er.de>
This hard-coded GCinterval value makes any change to the macro defining the GCinterval ineffective
when the GC runs.

For e.g., if I reduce the GC from once an hour to once every 3 minutes, the GC runs every 3 minutes,
but the mintime is set to the start of the next hour, which wipes everything from the overTime data.

I believe that this change is safe and can be merged and shipped to existing users because the
GCinterval value is not configurable and users who don't build the PiHole version that they are
running will not be affected by this change.

Signed-off-by: Siddharth Kannan <mail@siddharthkannan.in>
Signed-off-by: Christian König <ckoenig@posteo.de>
Reload blocking mode independently of incoming queries
Compiling FTL on MIPS platform would fail with the following error:
/home/git/FTL/src/database/../shmem.h:57:41: error: expected declaration specifiers or ‘...’ before numeric constant
   57 | ASSERT_SIZEOF(countersStruct, 240, 240, 240);
This error originates from static_assert.h, where there are no definition of ASSERT_SIZEOF and STATIC_ASSERT for the mips architecture.
Support added in this commit.
Signed-off-by: abesnier <besnier_antoine@yahoo.fr>
Signed-off-by: abesnier <besnier_antoine@yahoo.fr>
…interval

 Replace reference to hard-coded default GCinterval value
Signed-off-by: Christian König <ckoenig@posteo.de>
Be more precise about escaping invalid characters in queries
Adjust query type counters when importing from the database
Signed-off-by: DL6ER <dl6er@dl6er.de>
Added support for compilation on MIPS platform
…e we subtract from the previous reply counter when setting a new status

Signed-off-by: DL6ER <dl6er@dl6er.de>
Signed-off-by: DL6ER <dl6er@dl6er.de>
initOverTime calculate's the last slot timestamp with the assumption that `OVERTIME_INTERVAL` will
allow atleast `OVERTIME_SLOTS` number of slots between now and the last slot in the current
hour. This leads it to calculate the timestamp of the last slot, and then assign timestamps to each
slot counting down from that timestamp.

This logic is not compatible with changes in the OVERTIME_INTERVAL and the MAXLOGAGE variables. Here
is a demonstration of a case where this can be shown clearly:

```
OVERTIME_INTERVAL = 60s (Each slot stores 1 minute of data)
MAXLOGAGE = 600s (Store date for only 10 minutes)
OVERTIME_SLOTS = 11 (MAXLOGAGE/OVERTIME_INTERVAL)
```

- Current time: 10:35
- Last slot time: 10:59:30
- First slot time: 10:48:30

Note that this logic *does* work as expected with the default minimum of 1 hour for the MAXLOGAGE
configuration variable.

Signed-off-by: Siddharth Kannan <mail@siddharthkannan.in>
Fix initOverTime's calculation of last slot timestamp
Signed-off-by: DL6ER <dl6er@dl6er.de>
Co-authored-by: yubiuser <ckoenig@posteo.de>
yubiuser and others added 3 commits April 2, 2022 22:41
Signed-off-by: DL6ER <dl6er@dl6er.de>
Fix overtime computation logic broken in PR #1312
@PromoFaux
Copy link
Member Author

PromoFaux commented Apr 20, 2022

We can ignore the deploy fail in this case - it's because the release dir does not exist on the remote server - we don't need the binaries there anyway

Ignore this - I created the DIR

@dschaper dschaper self-requested a review April 20, 2022 18:31
@PromoFaux PromoFaux merged commit 4bb0bfe into master Apr 20, 2022
@PromoFaux PromoFaux deleted the release/v5.15 branch April 20, 2022 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants