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

Wordpress 6.6.2 upgrade #234

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Wordpress 6.6.2 upgrade #234

wants to merge 5 commits into from

Conversation

alithethird
Copy link
Contributor

@alithethird alithethird commented Jul 31, 2024 •
edited
Loading

Just upgraded the Wordpress version to 6.6.2

Overview

Upgraded Wordpress version to 6.6.2 as a response to the issue

Rationale

Never and shinier Wordpress.

Juju Events Changes

Module Changes

Library Changes

Checklist

@alithethird alithethird added the trivial This is a trivial PR label Jul 31, 2024
@alithethird alithethird requested a review from a team as a code owner July 31, 2024 14:12
@alithethird alithethird linked an issue Jul 31, 2024 that may be closed by this pull request
Requesting 6.5 build #218
Open
arturo-seijas
arturo-seijas previously approved these changes Jul 31, 2024
View reviewed changes
weiiwang01
weiiwang01 previously approved these changes Jul 31, 2024
View reviewed changes
@weiiwang01
Copy link
Collaborator

I ran the test locally and it seems like there's a problem with the launchpad wordpress plugin. That plugin hasn't been upgraded for a while; I think we might need to put this on hold.

2024-08-01T05:18:13.763Z [wordpress] [Thu Aug 01 05:18:13.763761 2024] [php7:error] [pid 181] [client 172.16.0.2:58743] PHP Fatal error:  Uncaught Error: Call to a member function query() on null in /var/www/html/wp-includes/widgets.php:2090\nStack trace:\n#0 /var/www/html/wp-includes/class-wp-hook.php(324): wp_check_widget_editor_deps()\n#1 /var/www/html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters()\n#2 /var/www/html/wp-includes/plugin.php(517): WP_Hook->do_action()\n#3 /var/www/html/wp-content/plugins/openid/common.php(764): do_action()\n#4 /var/www/html/wp-content/plugins/openid/common.php(746): openid_page()\n#5 /var/www/html/wp-content/plugins/openid/consumer.php(59): openid_repost()\n#6 /var/www/html/wp-content/plugins/openid/consumer.php(177): openid_redirect()\n#7 /var/www/html/wp-content/plugins/wordpress-launchpad-integration/openid-launchpad.php(173): openid_start_login()\n#8 /var/www/html/wp-includes/class-wp-hook.php(326): openid_launchpad_wp_authenticate()\n#9 /var/www/html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters()\n#10 /var/www/html/wp-includes/plugin.php(565): WP_Hook->do_action()\n#11 /var/www/html/wp-inclu in /var/www/html/wp-includes/widgets.php on line 2090, referer: http://10.1.75.149/wp-login.php?redirect_to=http%3A%2F%2F10.1.75.149%2Fwp-admin%2F&reauth=1

@alithethird alithethird dismissed stale reviews from arturo-seijas and weiiwang01 via 9c96b5b November 6, 2024 04:02
@alithethird alithethird changed the title Wordpress 6.5.3 upgrade Wordpress 6.6.2 upgrade Nov 6, 2024
nrobinaubertin
nrobinaubertin previously approved these changes Nov 6, 2024
View reviewed changes
cbartz
cbartz previously approved these changes Nov 6, 2024
View reviewed changes
@alithethird alithethird dismissed stale reviews from cbartz and nrobinaubertin via 0bca508 November 13, 2024 07:27
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Dec 9, 2024

Test coverage for c53b0d6

Name                Stmts   Miss Branch BrPart  Cover   Missing
---------------------------------------------------------------
src/charm.py          557     35    174     34    91%   162, 204-207, 299->302, 414, 473->477, 627, 658, 664, 681-685, 715, 750-751, 811, 820, 924->937, 936, 938, 943-944, 1004, 1024, 1031, 1121, 1130, 1142, 1163, 1172, 1191, 1195, 1224, 1265, 1397, 1419, 1426->1428, 1471->exit, 1483, 1493, 1531
src/cos.py             20      0      0      0   100%
src/exceptions.py      16      1      2      1    89%   41
src/state.py           33      0      2      0   100%
src/types_.py          15      0      0      0   100%
---------------------------------------------------------------
TOTAL                 641     36    178     35    91%

Static code analysis report

Run started:2024-12-09 03:31:46.911172

Test results:
  No issues identified.

Code scanned:
  Total lines of code: 4647
  Total lines skipped (#nosec): 1
  Total potential issues skipped due to specifically being disabled (e.g., #nosec BXXX): 0

Run metrics:
  Total issues (by severity):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
  Total issues (by confidence):
  	Undefined: 0
  	Low: 0
  	Medium: 0
  	High: 0
Files skipped (0):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arturo-seijas arturo-seijas arturo-seijas left review comments

@weiiwang01 weiiwang01 weiiwang01 left review comments

@nrobinaubertin nrobinaubertin nrobinaubertin left review comments

@cbartz cbartz cbartz left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Libraries: Out of sync trivial This is a trivial PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Requesting 6.5 build
6 participants
@alithethird @weiiwang01 @cbartz @nrobinaubertin @arturo-seijas @yanksyoon