Commit Graph

818 Commits

Author SHA1 Message Date
Kai Wagner
132fdccfd7 Merge pull request #273 from capri1989/PG-457
PG-457: added vim code annotation config example
2022-06-28 12:03:15 +02:00
Kai Wagner
ebf940a3c0 PG-457: added vim code annotation config example
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-06-28 09:06:18 +02:00
Kai Wagner
f73f3fec09 Merge pull request #270 from Naeem-Akhter/PG-452
PG-452: Remove redundant workflow from PGSM GH main branch.
2022-06-23 12:50:59 +02:00
Naeem Akhter
28d194a26a PG-452: Remove redundant workflow from PGSM GH main branch.
Removed following workflows from main branch.
postgresql-14-package.yml
postgresql-13-package.yml
postgresql-12-package.yml
postgresql-11-package.yml
2022-06-23 13:10:45 +05:00
Ibrar Ahmed
15c73407c3 PG-453: Normalize query does not work with INSERT statements.        … (#268)
PG-453: Normalize query does not work with INSERT statements.                  

The commit fixes the issue similar to pg_stat_statements. Jumble query skips in    
case of INSERT statements to avoid the duplicate queryid, but in another      
commit we already solved that problem with another way.
2022-06-22 11:53:42 +05:00
Ibrar Ahmed
2ba1960c2f Merge pull request #266 from EngineeredVirus/main
[PG-159] pg_stat_monitor: Bucket start time should be aligned with fi…
2022-06-22 00:00:00 +05:00
Kai Wagner
d131040845 Merge pull request #267 from capri1989/PG-437
PG-437: added the how we work section for OneFlow
2022-06-21 10:30:49 +02:00
Kai Wagner
ef24c8c324 PG-437: added the how we work section for OneFlow
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-06-21 08:38:54 +02:00
Hamid Akhtar
7efc3fa50e [PG-159] pg_stat_monitor: Bucket start time should be aligned with fix number of seconds
The buckets are now created with the start time a modulus of the bucket time size.
So if we have a 10 second bucket, the start times would reflect that:
- Bucket1: 00:00:00
- Bucket2: 00:00:10
- Bucket3: 00:00:20
...

Previously, the start time of the bucket was aligned with the first query that
arrives in that bucket. However, now the behaviour is changed. So, even if the
first query for bucket 2 arrives at 00:00:13, the start time would still be set
to 00:00:10.

This change now makes the bucketing separated out by fixed time windows so that
external applications can easily consume that data and chart it.

Also, as part of this change, locking of pgss is updated now and extended
to last the bucket related changes.
2022-06-20 17:53:43 +05:00
Ibrar Ahmed
86978bc4ce Merge pull request #265 from EngineeredVirus/main
Merging the REL_1_STABLE branch onto the main branch.
2022-06-20 17:22:47 +05:00
Hamid Akhtar
053f1d6e56 [PG-436] Completing the merge process by resolving feedback received during
the review process. Updating version to 1.1.0-dev.
2022-06-20 15:53:20 +05:00
Hamid Akhtar
cd06be62fd [PG-436] Merging REL_1_STABLE on to the main branch.
Merge remote-tracking branch 'origin/REL_1_STABLE'
2022-06-19 13:42:10 +05:00
Ibrar Ahmed
1f9b1feff6 PG-300: Fix potential duplicate queryid issue. (#262)
There is some potential problem where there is a chance
that we got a duplicate queryid. This patch will eliminate
that problem.
2022-06-17 16:32:55 +05:00
Kai Wagner
454b35e6e1 Merge pull request #260 from nastena1606/PG-370-Release-notes-RC1
PG-370 Release notes PGSM 1.0.0-rc1
2022-06-15 16:58:09 +02:00
Ibrar Ahmed
1ff4891191 PG-439: Remove warning of comparison of unsigned enum expression. (#257) 2022-06-14 18:04:43 +05:00
Anastasia Alexadrova
05df7f0e8e PG-370 Release notes PGSM 1.0.0-rc1 2022-06-14 13:07:01 +03:00
Kai Wagner
55ba2deee1 Merge pull request #259 from puneet0191/PMM-9951-4
PMM-9951 moving integration tests to push
2022-06-13 08:33:00 +02:00
puneet0191
ced7aef0e4 PMM-9951 moving integration tests to push 2022-06-09 18:23:50 +05:30
Puneet Kala
3d64931192 PMM-9951 run the workflow on push temp (#258) 2022-06-09 16:18:18 +05:00
Ibrar Ahmed
d5ce83738f PG-362: pgsm_normalized_query needs to be 0 by default. (#253)
pg_stat_monitor.pgsm_normalized_query needs to be 0 by default because
people will get Query examples. This is one of the main user-faced
advantages of pgsm over pgss (pg_stat_statements).
2022-06-09 16:13:58 +05:00
Ibrar Ahmed
1b89fa1814 PG-289: Remove ‘for’ loop initial declarations. (#256) 2022-06-09 16:13:43 +05:00
Kai Wagner
5194a108b3 Merge pull request #252 from capri1989/PG-438
PG-438: update README.md to point to the stable releases instead of a branch
2022-06-07 11:42:25 +02:00
Kai
3c0d19c72f PG-438: update README.md to point to the stable releases instead of a branch
Signed-off-by: Kai <kai.wagner@percona.com>
2022-06-07 10:11:08 +02:00
Puneet Kala
4f056813e0 PMM-9951 Checking with forked repo and custom branch (#251)
PMM-9951 Fix for forked Repo

PMM-9951 Checking with forked repo and custom branch

PMM-9951 fix the branch

PMM-9951 testing the github properties

PMM-9951 testing

PMM-9951 testing the changes

PMM-9951 changing permissions

PMM-9951 handle the repo

PMM-9951 permissions check

PMM-9951 permissions check

PMM-9951 remove not needed ones

PMM-9951 checks permissions

PMM-9951 testing

PMM-9951 testing another workflow

PMM-9951 test changes

PMM-9951 Final Push with all changes

PMM-9951
2022-06-03 16:11:15 +05:00
Puneet Kala
9f7aac6810 PMM-9951 Adding PGSM+PMM Integration tests (#249) 2022-06-01 16:53:58 +05:00
Kai Wagner
f046df0351 Merge pull request #246 from nastena1606/PG-432-Fix-doc-GH-action-1
PG-432 Fixed branch name of a  GH action
2022-06-01 10:55:08 +02:00
Anastasia Alexadrova
ce9eca0fc6 PG-432 Fixed branch name of a GH action
modified:   .github/workflows/doc-build.yml
2022-06-01 11:21:03 +03:00
Kai Wagner
95eb25e15e Merge pull request #227 from capri1989/PG-416
PG-416: added PostgreSQL14 package build workflow
2022-05-31 08:17:59 +02:00
Kai Wagner
cb454113b7 Merge pull request #234 from capri1989/main-PG-427
DISTPG-427: replaced return with goto exit to not break other extensions
2022-05-31 08:17:21 +02:00
Ibrar Ahmed
8330ecdaf2 README Update. (#244)
Co-authored-by: Ibrar Ahmed <“ibrar.ahmed@percona.com”>
1.0.1
2022-05-26 15:04:13 -04:00
Ibrar Ahmed
4a2dd52bc0 README Update. (#243)
Update the coverage badge repo.

Co-authored-by: Ibrar Ahmed <“ibrar.ahmed@percona.com”>
2022-05-26 14:03:39 -04:00
Ibrar Ahmed
b194a57312 Merge pull request #242 from ibrarahmad/MAIN-REG
PG-382: Adjust the maximum value for histogram buckets.
2022-05-26 22:44:36 +05:00
Ibrar Ahmed
774488f309 PG-382: Adjust the maximum value for histogram buckets.
Fix regression test cases.
2022-05-26 17:22:29 +00:00
Ibrar Ahmed
4bf7b80c8a Merge pull request #221 from ibrarahmad/PG-382
PG-382: Adjust the maximum value for histogram buckets.
2022-05-26 21:39:07 +05:00
Ibrar Ahmed
8c8773fd17 Merge pull request #239 from capri1989/PG-420
PG-420: PGSM-1.0.1 release notes
2022-05-26 21:38:23 +05:00
Kai Wagner
8586816194 PG-424: bump version to 1.0.1 (#235)
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-05-25 11:43:11 -04:00
Kai Wagner
c6f3e76310 Merge pull request #236 from ibrarahmad/REL1_0_STABLE
PG-382: Adjust the maximum value for histogram buckets.
2022-05-25 17:05:15 +02:00
Kai Wagner
de9a915a97 PG-420: PGSM-1.0.1 release notes
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-05-25 15:40:02 +02:00
Kai Wagner
1ea4fc1454 Merge pull request #238 from capri1989/stable-PG-375
PG-375: only run code coverage test on merge
2022-05-25 13:24:20 +02:00
Kai Wagner
6d2a81ebdd PG-375: only run code coverage test on merge
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-05-25 12:04:45 +02:00
Kai Wagner
71b70dac7f Merge pull request #237 from capri1989/PG-375
PG-375: only run code coverage test on merge
2022-05-25 11:51:25 +02:00
Ibrar Ahmed
13eedd579e PG-382: Adjust the maximum value for histogram buckets.
There was no maximum limit set for the number of maximum histograms
bucket, which can lead to a crash in case of higher value.

PG-382: Adjust the maximum value for histogram buckets.

Fix the regression issue related to GUC and set the maximum
buckets value correctly.

PG-382: Adjust the maximum value for histogram buckets.

Fix the TAP test cases.
2022-05-24 17:50:07 +00:00
Ibrar Ahmed
f60d3422df PG-382: Adjust the maximum value for histogram buckets.
There was no maximum limit set for the number of maximum histograms
bucket, which can lead to a crash in case of higher value.

PG-382: Adjust the maximum value for histogram buckets.

Fix the regression issue for PostgreSQL-12.

PG-382: Adjust the maximum value for histogram buckets.

Fix the TAP test cases.
2022-05-24 17:15:03 +00:00
Kai Wagner
233a578796 PG-375: only run code coverage test on merge
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-05-24 14:02:48 +02:00
Kai Wagner
cc70b571ae Merge pull request #233 from ibrarahmad/REL1_0_STABLE
PG-417: Replace 'CREATE OR REPLACE' with 'CREATE' for extension.
2022-05-23 13:24:08 +02:00
Kai Wagner
23859b5a01 Merge pull request #232 from ibrarahmad/REMOTE-PG-417
PG-417: Replace 'CREATE OR REPLACE' with 'CREATE' for extension.
2022-05-23 13:23:59 +02:00
Kai Wagner
9789af7dfc DISTPG-427: replaced return with goto exit to not break other extensions
Signed-off-by: Kai Wagner <kai.wagner@percona.com>
2022-05-23 08:28:25 +02:00
Ibrar Ahmed
c94f264911 Merge pull request #228 from capri1989/DISTPG-427
DISTPG-427: replaced return with goto exit to not break other extensions
2022-05-22 10:03:01 +05:00
Ibrar Ahmed
1b995c768d PG-417: Replace 'CREATE OR REPLACE' with 'CREATE' for extension.
While analyzing the pg_stat_monitor installation scripts I found several
vulnerabilities. pg_stat_monitor uses CREATE OR REPLACE to install its
functions which is a security hazard. An attacker can precreate the functions
have a superuser install the extension and after installation the attacker
can switch out the function with a malicious version since he would still
be the owner of the function. Instead of CREATE OR REPLACE the installation
script should use plain CREATE to prevent this attack.

For reference
https://www.postgresql.org/docs/current/extend-extensions.html#EXTEND-EXTENSIONS-SECURITY
https://github.com/timescale/pgspot
2022-05-22 03:49:03 +00:00
Ibrar Ahmed
a59bca088a PG-417: Replace 'CREATE OR REPLACE' with 'CREATE' for extension.
While analyzing the pg_stat_monitor installation scripts I found several
vulnerabilities. pg_stat_monitor uses CREATE OR REPLACE to install its
functions which is a security hazard. An attacker can precreate the functions
have a superuser install the extension and after installation the attacker
can switch out the function with a malicious version since he would still
be the owner of the function. Instead of CREATE OR REPLACE the installation
script should use plain CREATE to prevent this attack.

For reference
https://www.postgresql.org/docs/current/extend-extensions.html#EXTEND-EXTENSIONS-SECURITY
https://github.com/timescale/pgspot
2022-05-22 03:44:32 +00:00