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.

Sign up for GitHub

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

Jump to bottom

HookLog fails to add records for PG because invalid syntax #5072

Closed
7 tasks
christian7007 opened this issue Sep 9, 2020 · 0 comments
Closed
7 tasks

HookLog fails to add records for PG because invalid syntax #5072

christian7007 opened this issue Sep 9, 2020 · 0 comments
Assignees
@christian7007
Labels
Category: Core & System Priority: Normal Status: Accepted Type: Bug

Comments

@christian7007
Copy link
Contributor

Description

MySQL ÌFNULL() function is not supported for PostgresSQL, this function is used in HookLog::add(). This makes adding a new record fails when PostgresSQL is used.

To Reproduce

  • Create a new hook.
  • Trigger the hook.

Expected behavior

The new hook log record is added correctly using PostgresSQL.

Details

  • Affected Component: oned
  • Version: development

Progress Status

  • Branch created
  • Code committed to development branch
  • Testing - QA
  • Documentation
  • Release notes - resolved issues, compatibility, known issues
  • Code committed to upstream release/hotfix branches
  • Documentation committed to upstream release/hotfix branches
@christian7007 christian7007 self-assigned this Sep 9, 2020
christian7007 pushed a commit to christian7007/docs that referenced this issue Sep 24, 2020
F OpenNebula/one#5072: update resolved issues
7c56bfc
rsmontero pushed a commit to OpenNebula/docs that referenced this issue Sep 24, 2020
F OpenNebula/one#5072: update resolved issues (#1299)
c62f252
tinova added a commit to OpenNebula/docs that referenced this issue Sep 24, 2020
@tinova
Revert "F OpenNebula/one#5072: update resolved issues (#1299)"
e7a8d91 
This reverts commit c62f252.
christian7007 pushed a commit to christian7007/docs that referenced this issue Sep 24, 2020
F OpenNebula/one#5072: update resolved issues
9ffcb9f
rsmontero pushed a commit that referenced this issue Sep 24, 2020
@rsmontero
B #5072: Make HookLog compatible with every DB backend (#241)
2cc5f13
rsmontero pushed a commit that referenced this issue Sep 24, 2020
@rsmontero
B #5072: Make HookLog compatible with every DB backend (#241)
cebd6c6 
(cherry picked from commit 2cc5f13)
rsmontero pushed a commit to OpenNebula/docs that referenced this issue Sep 24, 2020
F OpenNebula/one#5072: update resolved issues (#1302)
73a340e
rsmontero added a commit to OpenNebula/docs that referenced this issue Sep 24, 2020
@rsmontero
B OpenNebula/one#5072 OpenNebula/one#5081: update resolved issues
983637a
rsmontero pushed a commit that referenced this issue Oct 6, 2020
@rsmontero
B #5072: Make HookLog compatible with every DB backend (#241)
c02ed57 
(cherry picked from commit 7bd3185)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Oct 15, 2020
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
bf8a307 
…nNebula#241)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Oct 16, 2020
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
af8b752 
…nNebula#241)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Oct 16, 2020
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
a8adbe2 
…nNebula#241)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Nov 27, 2020
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
4bc1e59 
…nNebula#241)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Nov 27, 2020
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
a0371cb 
…nNebula#241)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Jan 4, 2021
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
95c04b4 
…nNebula#241)
atodorov-storpool pushed a commit to storpool/one that referenced this issue Jan 7, 2021
@atodorov-storpool
B OpenNebula#5072: Make HookLog compatible with every DB backend (Ope…
db853af 
…nNebula#241)
rsmontero pushed a commit that referenced this issue Jan 20, 2021
B #5072: Make HookLog compatible with every DB backend (#241)
7bd3185
@christian7007 christian7007 changed the title HookLog fails to add records for PG because invalid syntas HookLog fails to add records for PG because invalid syntax Mar 16, 2021
@christian7007 christian7007 mentioned this issue Mar 16, 2021
Closed
7 tasks
rsmontero pushed a commit that referenced this issue Apr 13, 2021
@rsmontero
B #5072: Make HookLog compatible with every DB backend (#241)
abd04c7 
(cherry picked from commit 7bd3185)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@christian7007 christian7007

Labels
Category: Core & System Priority: Normal Status: Accepted Type: Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@christian7007