Skip to content

Sync: BioPortal v6.1.0 release - #4

Merged
alexskr merged 92 commits into
ontoportal:masterfrom
ncbo:master
Jul 21, 2025
Merged

Sync: BioPortal v6.1.0 release#4
alexskr merged 92 commits into
ontoportal:masterfrom
ncbo:master

Conversation

@alexskr

@alexskr alexskr commented May 1, 2025

Copy link
Copy Markdown
Contributor

alexskr and others added 30 commits October 15, 2024 16:45
Develop to master merge, release v6.0.0
update path of the appliance version manifest
…aths, simplify pid default

* config.rb
  • move log_path derivation **after** user block;
  • default log_dir → File.expand_path("logs", Dir.pwd)   (never gem‑relative);
  • default pid_path → File.join(Dir.pwd, "ncbo_cron.pid") – drops /var/run heuristic.

* bin/ncbo_cron
  • mkdir_p the chosen log directory **before** Dante::Runner is built;
    abort with clear message if creation fails (no stdout fallback for now).
  • keep existing daemon behaviour; systemd units can still override
    --log and --pid.

* addreses unwanted behavior where ontologies_api writes to $GEM_HOME/gems/ncbo_cron/logs
fix(logging+pid): respect custom log_dir
- Renamed default log directory from 'logs' to 'log'
- Aligns with common Ruby/Rails conventions (e.g., log/scheduler.log)
- Helps reduce confusion across components like ontologies_api which already use 'log'
- Does not affect behavior if log_dir is explicitly set in config
Rename logs directory to log for consistency with standard layout
jvendetti and others added 29 commits June 25, 2025 14:08
Raise MissingPullLocationException when an ontology has no pullLocation
set, and catch it in the ontology pull loop to avoid logging a full
stack trace. Log a concise warning instead.
Develop -> Master merge, release v6.2.0
Setting $0 = "ncbo_cron" before `require` calls causes Ruby to misreport
errors as "cannot load such file -- ncbo_cron"
…ion-handling

Handle ontologies with missing pullLocation gracefully in pull job
Develop to Master merge, release v6.3.0
scripts in bin directory are not inteded to be executables when this gem
is included as a dependency
remove executable directives and add development dependencies
Add email notifications to CloudflareAnalytics job; support multiple hostnames
Develop to Master merge, release v6.4.1
@alexskr
alexskr merged commit 592d607 into ontoportal:master Jul 21, 2025
2 checks passed
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.

3 participants