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

Update staging instance with bug fixes #80

Open
maccraymer opened this issue Apr 25, 2020 · 19 comments
Open

Update staging instance with bug fixes #80

maccraymer opened this issue Apr 25, 2020 · 19 comments
Assignees
Labels
enhancement New feature or request

Comments

@maccraymer
Copy link

Please update the staging instance of the Registry with Florian’s bug fixes so I can test that all the issues have been resolved. Please do not update the public instance until I have finished testing the staging instance.

The ISO Control Body would appreciate if this could be given the highest priority right now as there are errors in the current Registry content that we are unable to correct until the bug fixes are tested and implemented. As it presently exists, the Registry is unusable without these bug fixes and the Excel importing function, and, worse, it continues to provide erroneous information to the public. Thanks!

@ronaldtse
Copy link
Collaborator

@maccraymer sorry we had to take the time to finish testing Florian's bug fixes with the existing database in order to deploy. Most issues are now verified.

@phuonghuynh will be deploying staging once we are able to do so.

@ronaldtse ronaldtse added the enhancement New feature or request label Apr 26, 2020
@phuonghuynh
Copy link
Contributor

phuonghuynh commented Apr 26, 2020

Staging updated to "master" branch now, which contains fixes from florianesser

List of PRs merged can be found here

@maccraymer
Copy link
Author

Thanks Ron & Phuong! Can this issue be closed? I probably should have just emailed you instead of making an “issue” out of this request.

@maccraymer
Copy link
Author

@ronaldtse & @phuonghuynh, can you please update both the public & staging instances of the registry with the latest master that includes all closed bug fixes and enhancements. The currently open issues are not as urgent and should not preclude the CB from starting to update the Registry content after getting an answer to my last comment in #92.

And should we keep this issue open for me to periodically request updating the staging and public instances? Rather than creating a new issue each time.

@phuonghuynh
Copy link
Contributor

@maccraymer staging has been updated with master, could you help to check it before I deploy master to public server?

@phuonghuynh
Copy link
Contributor

@maccraymer public site has been deployed

@maccraymer
Copy link
Author

@phuonghuynh I checked the staging instance and found another 500 error using the Details action after the RM accepts a clarification proposal (see #95).

@phuonghuynh phuonghuynh pinned this issue May 19, 2020
@ronaldtse
Copy link
Collaborator

This should have been resolved.

@phuonghuynh
Copy link
Contributor

@ronaldtse this issue should not be close to let maccraymer periodically request updates for servers.

And should we keep this issue open for me to periodically request updating the staging and public instances? Rather than creating a new issue each time.

@maccraymer
Copy link
Author

Please update both the public and staging instances with the latest master. The open bug issues are not critical for revising the existing content. Thanks!

@maccraymer
Copy link
Author

Just wondering when you will be able to update both the public and staging instances. A couple of CB members want to correct some of their current content before the end of the week. Thanks.

@ronaldtse
Copy link
Collaborator

@maccraymer we will be updating this today. Thanks!

@phuonghuynh
Copy link
Contributor

Updated both staging+production to master githash 67180d4

@maccraymer
Copy link
Author

Please update the public registry to the same version as the staging instance. The staging instance appears to be fixed for GML output but the public registry is still giving the 500 error.

@phuonghuynh
Copy link
Contributor

Yes, i will revert changes made for Java8 to Java7, this will fix Linux server GML view

@maccraymer
Copy link
Author

Thanks @phuong! Maybe Java7 will also fix the new issue #112 I just reported (500 error when deleting a user). I only get the error with the public instance, not the staging one. Let me know here when you have reverted and I will check if the delete error is still present. Thanks.

@maccraymer
Copy link
Author

The staging registry appears to be corrupted after much testing of different kinds. Can you please reinstall the database. Please install the current public database for the staging registry so we have an up to date version. But please keep a backup of the older versions in case we need to revert.

The public registry may also be corrupted (at least the one item 138). What backups are available for the public registry's database? Can we step back in time and check older versions of the database in the staging registry to see when the item might have become corrupted or if was corrupted all a long?

Thanks!

@phuonghuynh
Copy link
Contributor

@ronaldtse should we implement cloudwatch for this DB backup action?

@ronaldtse
Copy link
Collaborator

@phuonghuynh yes, let's:

  1. create a cloudwatch periodic job to perform the backup from production (we already have this, right)?
  2. update the staging site to restore the production data using cloudwatch periodically (every sunday?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants