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

SQL-based Fedora object persistence (Issue 1244) #132

Merged
merged 4 commits into from
Aug 30, 2019
Merged

SQL-based Fedora object persistence (Issue 1244) #132

merged 4 commits into from
Aug 30, 2019

Conversation

seth-shaw-unlv
Copy link
Contributor

DO NOT MERGE! Used for testing...

GitHub Issue: /Islandora/documentation/issues/1244

What does this Pull Request do?

Used to test /islandora-deprecated/ansible-role-fcrepo/pull/8 by using Fedora's MySQL-based object persistence storage instead of the default simple file one.

What's new?

  • Change: the requirements to point to the ansible-role-fcrepo branch we're testing
  • Added:
    • variables to tomcat8_java_opts necessary to use the MySQL storage
    • the fcrepo_persistence variable (set to jdbc-mysql)
    • fcrepo_db_password to the passwords config
    • Java roles (because idempotency for the Tomcat playbook is a good thing)
  • Does this change require documentation to be updated? Maybe if we keep parts
  • Does this change add any new dependencies? No.
  • Does this change require any other modifications to be made to the repository
    (ie. Regeneration activity, etc.)? No.
  • Could this change impact execution of existing code? No.

How should this be tested?

  • Apply the PR
  • vagrant up
  • vagrant ssh
  • mysql -u fcrepo -p fcrepo (password: islandora)
  • Ensure the fcrepo database is being populated
  • Smoke test that Fedora is working properly

Interested parties

@dannylamb, @whikloj, @Islandora-Devops/committers

@dannylamb
Copy link
Member

vagrant up

@dannylamb
Copy link
Member

dannylamb commented Aug 28, 2019

@seth-shaw-unlv I have no fcrepo database in mysql.

vagrant@claw:~$ mysql -u fcrepo -p fcrepo
Enter password: 
ERROR 1045 (28000): Access denied for user 'fcrepo'@'localhost' (using password: YES)

I logged in as root to confirm. What am I missing here?

@seth-shaw-unlv
Copy link
Contributor Author

@dannylamb, um, did you clear your external roles??

Also, can you find "Create Fedora DB (mysql)" in the ansible log? What does that say?

@seth-shaw-unlv
Copy link
Contributor Author

I guess since the error was with authorization; what does "Create Fedora DB User (mysql)" in the ansible log say?

@seth-shaw-unlv
Copy link
Contributor Author

I should also note that the password for this PR (islandora) overrides the default password provided by the role (fcrepo).

@dannylamb
Copy link
Member

dannylamb commented Aug 28, 2019 via email

@dannylamb
Copy link
Member

Looking good @seth-shaw-unlv

vagrant@claw:~$ mysql -u fcrepo -p fcrepo
Enter password: 
Reading table information for completion of table and column names
You can turn off this feature to get a quicker startup with -A

Welcome to the MySQL monitor.  Commands end with ; or \g.
Your MySQL connection id is 154
Server version: 5.7.27-0ubuntu0.18.04.1 (Ubuntu)

Copyright (c) 2000, 2019, Oracle and/or its affiliates. All rights reserved.

Oracle is a registered trademark of Oracle Corporation and/or its
affiliates. Other names may be trademarks of their respective
owners.

Type 'help;' or '\h' for help. Type '\c' to clear the current input statement.

mysql> DESCRIBE MODESHAPE_REPOSITORY;
+--------------+--------------+------+-----+-------------------+-----------------------------+
| Field        | Type         | Null | Key | Default           | Extra                       |
+--------------+--------------+------+-----+-------------------+-----------------------------+
| ID           | varchar(255) | NO   | PRI | NULL              |                             |
| LAST_CHANGED | timestamp    | NO   |     | CURRENT_TIMESTAMP | on update CURRENT_TIMESTAMP |
| CONTENT      | longblob     | NO   |     | NULL              |                             |
+--------------+--------------+------+-----+-------------------+-----------------------------+
3 rows in set (0.01 sec)

mysql> SELECT COUNT(*) FROM MODESHAPE_REPOSITORY;
+----------+
| COUNT(*) |
+----------+
|      461 |
+----------+
1 row in set (0.00 sec)

Copy link
Member

@dannylamb dannylamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@seth-shaw-unlv I sliced a 2.0.0 version of ansible-role-fcrepo. Update this PR's requirements.yml and I'll merge 👍

@seth-shaw-unlv
Copy link
Contributor Author

@dannylamb, mischief managed.

@dannylamb dannylamb merged commit 1a77544 into Islandora-Devops:dev Aug 30, 2019
whikloj pushed a commit to whikloj/claw-playbook that referenced this pull request Dec 18, 2019
* support Fedora object SQL-based persistence

* fix test branch version

* remove unnecessary tomcat8 JAVA_OPTS

* use ansible-role-fcrepo v.2.0.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants