g5k-api merge requestshttps://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests2017-10-24T14:13:01+02:00https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/44Bugs/#8379/tls client certs2017-10-24T14:13:01+02:00MARGERY DavidBugs/#8379/tls client certs*Created by: dmargery*
Update codebase so that it can directly request to the API using the local certs.*Created by: dmargery*
Update codebase so that it can directly request to the API using the local certs.https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/42Use X_FORWARDED_HOST only for base_uri(:in)2018-01-26T13:57:02+01:00MARGERY DavidUse X_FORWARDED_HOST only for base_uri(:in)*Created by: dmargery*
A commit I forgot in previous PR.*Created by: dmargery*
A commit I forgot in previous PR.https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/39Bug8030 rebased add job noop2017-10-27T17:37:57+02:00MARGERY DavidBug8030 rebased add job noop*Created by: f-didier*
*Created by: f-didier*
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/41Bugs/#8542: update our patched version of exhibit.js2017-09-14T16:59:12+02:00MARGERY DavidBugs/#8542: update our patched version of exhibit.js*Created by: dmargery*
Hello,
investigating bug #8542, I discovered we are using a version of exhibit.js that was patched to be able to work from our own servers rather than the public servers publishing the library.
This might b...*Created by: dmargery*
Hello,
investigating bug #8542, I discovered we are using a version of exhibit.js that was patched to be able to work from our own servers rather than the public servers publishing the library.
This might be linked to an old discussion about avoiding warning about getting resources from an http source on an https page.
This pull request updates the patch to serve the JavaScript files from the assets of g5k-api rather than from https://api.grid5000.fr, in order to be able to use g5k-api's webui through a proxy.
In the process, I fixed issues preventing me from testing my changes:
* an issue with exhibit not being able to decide whether the locale is fr or FR-fr by duplicating a locale.js
* an issue with the development environment not proxying requests to kadeployapi
* the reference to reims, no longer a Grid'5000 sitehttps://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/40Compute absolute urls using X-Forwarded-Host2017-09-13T09:28:07+02:00MARGERY DavidCompute absolute urls using X-Forwarded-Host*Created by: dmargery*
See bug #8536 for the context.
With this code, absolute urls of type :in (exposing how to reach g5k-api
resources) now only rely on the config file (base_uri_in) if no
X-Forwarded-Host, added by the request b...*Created by: dmargery*
See bug #8536 for the context.
With this code, absolute urls of type :in (exposing how to reach g5k-api
resources) now only rely on the config file (base_uri_in) if no
X-Forwarded-Host, added by the request by proxies, is present.https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/38Update to rspec-rails & rspec supporting rake 12.02017-09-01T15:19:46+02:00MARGERY DavidUpdate to rspec-rails & rspec supporting rake 12.0*Created by: dmargery*
Hello,
following my investigations of bug #8196, I discovered that while we stay with rspec2.6, we can't upgrade to rake >= 11.0.1
In this PR, I've updated rspec-rails and rails and updated the tests to hand...*Created by: dmargery*
Hello,
following my investigations of bug #8196, I discovered that while we stay with rspec2.6, we can't upgrade to rake >= 11.0.1
In this PR, I've updated rspec-rails and rails and updated the tests to handle the new version of rspec.
To complete the PR, all tests should migrate to the expect().to syntax rather than the .should syntax if we want to avoid deprecation warnings.
@f-didier : do you have many branches of the code with new tests that would also need migrated ?
David https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/37Bug#8489 api root path2017-09-05T09:51:46+02:00MARGERY DavidBug#8489 api root path*Created by: dmargery*
Add support for a X-Api-Root-Path header to support getting requests for the API at urls such as https://proxy.fed4fire.eu/proxies/grid5000
David*Created by: dmargery*
Add support for a X-Api-Root-Path header to support getting requests for the API at urls such as https://proxy.fed4fire.eu/proxies/grid5000
Davidhttps://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/34optimization: factor building the job description2017-04-03T14:36:12+02:00MARGERY Davidoptimization: factor building the job description*Created by: lnussbaum*
*Created by: lnussbaum*
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/33Fix for disk reservation (bug 7929)2017-04-03T11:40:33+02:00MARGERY DavidFix for disk reservation (bug 7929)*Created by: f-didier*
*Created by: f-didier*
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/32Modified the mechanism for URI construction - for test & development2016-12-12T15:43:15+01:00MARGERY DavidModified the mechanism for URI construction - for test & development*Created by: nirvanesque*
- Modified the mechanism for URI construction
- for test & development environments.
- Added spec tests also.*Created by: nirvanesque*
- Modified the mechanism for URI construction
- for test & development environments.
- Added spec tests also.https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/31Single commit for bug #7360 - added option job-key-from-file2017-07-18T13:54:48+02:00MARGERY DavidSingle commit for bug #7360 - added option job-key-from-file*Created by: nirvanesque*
- Single commit for bug #7360 - added option job-key-from-file as variable job_key_from_file + spec test
- other textual changes are cosmetic*Created by: nirvanesque*
- Single commit for bug #7360 - added option job-key-from-file as variable job_key_from_file + spec test
- other textual changes are cosmetichttps://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/30Bugs/#7464 - /servers pour jessie-v32016-11-02T11:06:40+01:00MARGERY DavidBugs/#7464 - /servers pour jessie-v3*Created by: nirvanesque*
Il y a 3 commits séparés:
--- inclusion de /clusters pour nancy, dans git-interne (supprimé la fausse annulation),
--- ajoute de /servers pour nancy, dans git-interne
--- code-logique et tests unitaires pour /s...*Created by: nirvanesque*
Il y a 3 commits séparés:
--- inclusion de /clusters pour nancy, dans git-interne (supprimé la fausse annulation),
--- ajoute de /servers pour nancy, dans git-interne
--- code-logique et tests unitaires pour /servers
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/29Added spec test to check for link /servers for sites containing servers2016-10-26T15:24:21+02:00MARGERY DavidAdded spec test to check for link /servers for sites containing servers*Created by: nirvanesque*
Added spec test to check for link /servers for sites containing servers (e.g. nancy in test repository)
Note : The functionality (to create relevant links for servers, if present in a site) was already impleme...*Created by: nirvanesque*
Added spec test to check for link /servers for sites containing servers (e.g. nancy in test repository)
Note : The functionality (to create relevant links for servers, if present in a site) was already implemented correctly earlier (PR for bug/#7301)
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/28bug #7364 - Changed item links in /sites from deployments to deployment …2016-10-26T10:36:49+02:00MARGERY Davidbug #7364 - Changed item links in /sites from deployments to deployment …*Created by: nirvanesque*
Changed item links in /sites from deployments to deployment to coincide with v4.0 of g5k-api.
Added spec test to verify.
*Created by: nirvanesque*
Changed item links in /sites from deployments to deployment to coincide with v4.0 of g5k-api.
Added spec test to verify.
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/27Bugs/#73012016-10-25T10:13:02+02:00MARGERY DavidBugs/#7301*Created by: nirvanesque*
Consists of 2 commits as follows:
1. added /servers to site nancy in test repository : changes to git-interne and relevant spec tests due to changes in hash values.
2. added logic & tests for /servers functiona...*Created by: nirvanesque*
Consists of 2 commits as follows:
1. added /servers to site nancy in test repository : changes to git-interne and relevant spec tests due to changes in hash values.
2. added logic & tests for /servers functionality : new controller & spec files introduced + changes to routes.
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/26Cleanup internal git2016-10-25T10:13:05+02:00MARGERY DavidCleanup internal git*Created by: nirvanesque*
cleared any accidental remove of nancy clusters in test repository
*Created by: nirvanesque*
cleared any accidental remove of nancy clusters in test repository
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/25Bugs/#5856 - Cleared accidental remove of sites/nancy/clusters/2016-10-25T10:13:11+02:00MARGERY DavidBugs/#5856 - Cleared accidental remove of sites/nancy/clusters/*Created by: nirvanesque*
Cleared accidental remove of sites/nancy/clusters/ in bug/#5856
*Created by: nirvanesque*
Cleared accidental remove of sites/nancy/clusters/ in bug/#5856
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/20Bugs/#63632016-09-07T08:58:04+02:00MARGERY DavidBugs/#6363*Created by: nirvanesque*
"Second commit for bug #6363 with all unit tests successful + updated reference-repository"
*Created by: nirvanesque*
"Second commit for bug #6363 with all unit tests successful + updated reference-repository"
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/23Bugs/#7301 - to expose /servers on each site2016-10-24T16:52:03+02:00MARGERY DavidBugs/#7301 - to expose /servers on each site*Created by: nirvanesque*
Added /servers with 2 servers in site nancy + cleared git remove for /clusters in nancy - accidentally made in earlier commit.
*Created by: nirvanesque*
Added /servers with 2 servers in site nancy + cleared git remove for /clusters in nancy - accidentally made in earlier commit.
https://gitlab.inria.fr/grid5000/g5k-api/-/merge_requests/21Fix Grid'5000 bug #6693 by adding em-postgresql-adapter gem2016-03-16T16:18:15+01:00MARGERY DavidFix Grid'5000 bug #6693 by adding em-postgresql-adapter gem*Created by: clement-parisot*
See Bugs/#6693 on Grid5000:
API-Devel servers are down.
In the thin.log:
```
/opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activerecord-3.0.10/lib/active_record/connection_adapters/abstract/connection_...*Created by: clement-parisot*
See Bugs/#6693 on Grid5000:
API-Devel servers are down.
In the thin.log:
```
/opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activerecord-3.0.10/lib/active_record/connection_adapters/abstract/connection_specification.rb:71:in `rescue in establish_connection': Please install the em_postgresql adapter: `gem install
activerecord-em_postgresql-adapter` (no such file to load -- active_record/connection_adapters/em_postgresql_adapter) (RuntimeError)
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activerecord-3.0.10/lib/active_record/connection_adapters/abstract/connection_specification.rb:68:in `establish_connection'
from /opt/local/g5k-api/config/initializers/oar.rb:17:in `<top (required)>'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:235:in `load'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:235:in `block in load'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:227:in `load_dependency'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/activesupport-3.0.10/lib/active_support/dependencies.rb:235:in `load'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/engine.rb:201:in `block (2 levels) in <class:Engine>'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/engine.rb:200:in `each'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/engine.rb:200:in `block in <class:Engine>'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/initializable.rb:25:in `instance_exec'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/initializable.rb:25:in `run'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/initializable.rb:50:in `block in run_initializers'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/initializable.rb:49:in `each'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/initializable.rb:49:in `run_initializers'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/application.rb:134:in `initialize!'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/railties-3.0.10/lib/rails/application.rb:77:in `method_missing'
from /opt/local/g5k-api/config/environment.rb:20:in `<top (required)>'
from /opt/local/g5k-api/config.ru:3:in `require'
from /opt/local/g5k-api/config.ru:3:in `block in <main>'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/rack-1.2.4/lib/rack/builder.rb:46:in `instance_eval'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/rack-1.2.4/lib/rack/builder.rb:46:in `initialize'
from /opt/local/g5k-api/config.ru:1:in `new'
from /opt/local/g5k-api/config.ru:1:in `<main>'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/thin-1.2.11/lib/rack/adapter/loader.rb:36:in `eval'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/thin-1.2.11/lib/rack/adapter/loader.rb:36:in `load'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/thin-1.2.11/lib/thin/controllers/controller.rb:181:in `load_rackup_config'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/thin-1.2.11/lib/thin/controllers/controller.rb:71:in `start'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/thin-1.2.11/lib/thin/runner.rb:185:in `run_command'
from /opt/local/g5k-api/vendor/bundle/ruby/1.9.1/gems/thin-1.2.11/lib/thin/runner.rb:151:in `run!'
from /opt/local/g5k-api/bin/g5k-api.rb:32:in `<top (required)>'
from <internal:lib/rubygems/custom_require>:29:in `require'
from <internal:lib/rubygems/custom_require>:29:in `require'
from /usr/bin/g5k-api:2:in `<main>'
```
In Gemfile of g5k-api I find this commented line.
```
#gem 'em-postgresql-adapter', :git => 'git://github.com/leftbee/em-postgresql-adapter.git', :branch => 'pre-3_1'
```
Maybe need to be uncommented ?