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

Temporary Update for Post v0.1.22 Changes #110

Closed
JoelProminic opened this issue Feb 5, 2024 · 8 comments
Closed

Temporary Update for Post v0.1.22 Changes #110

JoelProminic opened this issue Feb 5, 2024 · 8 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@JoelProminic
Copy link
Contributor

I have a few changes I want to make in demo-tasks. However, it looks like updating past v0.1.22 will require some extra changes in Super.Human.Installer (#99). I'll plan to manually copy these changes if we get stuck on v0.1.22:

STARTcloud/hcl_roles#1
STARTcloud/hcl_roles#2
STARTcloud/hcl_roles#3

@JoelProminic JoelProminic added the bug Something isn't working label Feb 5, 2024
@JoelProminic JoelProminic added this to the v0.8.22 milestone Feb 5, 2024
@JoelProminic JoelProminic self-assigned this Feb 5, 2024
JoelProminic added a commit that referenced this issue Feb 6, 2024
@JoelProminic
Copy link
Contributor Author

@Aszusz, I made another update (on the development branch) to fix the Windows scripts. I confirmed that the updates do not break the actions on macOS. Please test this on Windows when you have a chance.

@Aszusz
Copy link
Collaborator

Aszusz commented Feb 7, 2024

I created new build and tested on Windows. It seems to fail much later in the process (but still complains about backslashes). It also prints insanely large log to the Moonshine IDE console. Attaching the log.
conversion-log.txt

@JoelProminic
Copy link
Contributor Author

The lengthy log is because the Maven build is doing some initial setup for the project.

The ZIP message is just a warning - my code update properly detects the error code for this case and continues the action, so this seems to be working properly. I could redirect STDERR for this command so that it doesn't show up in the error ouptut.

The actual error looks like a permission issue:

: [ERROR] Failed to execute goal org.openntf.maven:nsfodp-maven-plugin:3.8.1:generate-metadata (default-generate-metadata) on project nsf-moonshine-domino: Execution default-generate-metadata of goal org.openntf.maven:nsfodp-maven-plugin:3.8.1:generate-metadata failed: java.nio.file.AccessDeniedException: /tmp/nsfodp/20240207082422/nsfs/nsf-moonshine/odp/Code/Java/demo -> [Help 1]

@JoelProminic
Copy link
Contributor Author

@Aszusz, if you haven't already, please check the ownership on these files. Both the owner and group should be java_user:

ls -lh /tmp/nsfodp/20240207082422/nsfs/nsf-moonshine/odp/Code/Java/demo

This should have been fixed by this line. We confirmed that the line looks like this:

java_user@dev-1201:/opt/domino/scripts$ grep "chown" /opt/nsfodp/run_nsfodp.sh 
sudo chown -R java_user:java_user .

If you continue having problems, please send me a copy of your zipped project, and I'll do more testing with this. If you want to try to test the script yourself, you can run:

sudo su - java_user
bash
/opt/nsfodp/run_nsfodp.sh /tmp/restinterface/upload/2FD6B26F-7EA2-EDB4-B9A9-83F48204A978.zip

You also mentioned some agent errors. If you did not successfully deploy the database in the above step, that could explain the error. You can debug the database access like this:

sudo su - java_user
bash
java -jar CheckDatabase.jar mars/planets PieChartNsf.nsf

If you are still having errors, please send me the errors.

JoelProminic added a commit that referenced this issue Feb 8, 2024
@JoelProminic
Copy link
Contributor Author

I made some further tweaks to the above scripts:

  • use java_group insertion parameter
  • use insertion parameters for domino user and group
  • Added 2>&1 to unzip command so that warning is displayed as part of the normal output rather than as a separate error.

I wouldn't expect any of these changes to affect the behavior of the script significantly, though. @Aszusz, let me know if you continue having errors.

FYI, @piotrzarzycki21 reported that I should switch back to the master branch. I noticed that master didn't have the changes from my previous commit, but it looks like the development branch is removed now. I committed the scripts to master.

JoelProminic added a commit that referenced this issue Feb 8, 2024
@JoelProminic
Copy link
Contributor Author

I also pushed some updates for the other pending issues.

  • Added import_json_documents.sh script and capabilities
  • Updated REST API capabilities. The other file updates are not visible through the REST API yet - I need to make updates on the REST API itself next.

@JoelProminic
Copy link
Contributor Author

The provisioner updates are done, but I need to make an update to the REST Interface to return the new properties. This doesn't necessarily need to block a release.

@JoelProminic
Copy link
Contributor Author

We are planning an 0.9.0 release tomorrow, so I'll revisit the REST-Interface updates later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants