These are chat archives for allegro/ralph

23rd
Feb 2017
PdeNiet
@PdeNiet
Feb 23 2017 10:14
Hey guys, in the 'support' portion of Ralph3 all items have Status 'new'. I can't add new status and the database refers to an id-value. Is this hard coded?
PdeNiet
@PdeNiet
Feb 23 2017 10:51
bleh tried to edit /opt/ralph/ralph-core/lib/python3.4/site-packages/ralph/supports/migrations/0001_initial.py
but didnt work
PdeNiet
@PdeNiet
Feb 23 2017 10:56
@vi4m You mentioned prod_ldap.txt for following the ldap tutorial, where can you find this file?
nvm was overlooking it ':(
PdeNiet
@PdeNiet
Feb 23 2017 12:25
any thoughs with regards to adding more options under Supports>Supports>[item]>Status ?
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 13:48
@SixArt46 don’t bother with version 2.x - 3.x is much better :)
@rbuckland the status is:
@rbuckland beast is still generic API client which consumes /api endpoints, some people use it for list/create/update api endpoints
@rbuckland ralph-cli is more narrow focuses right now - don’t consume all endpoints, we only implemented selected parts, such as: updating asset componets (discovery), and that’s it for now
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 13:54
ralph-cli basically can only update server components, using scripts (ilo and idrac is provided)
Zrzut ekranu 2017-02-23 o 14.53.13.png
it’s open to discuss which route(generic api client? / go app?) is appropriate and easier to develop
we’re waiting for your feedback here
@rbuckland speaking about the pure python beast api client, show me where installation fails
@damian1976 how’s your scrooge production doing?
i’m goint to do some keynote / give a speach about Ralph in open source, need some input here ;)
PdeNiet
@PdeNiet
Feb 23 2017 15:02
@vi4m have you got any points for me with regards to adding more 'status' to a support entry?
*pointers
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:21
@PatrickdeNiet just a sec.
PdeNiet
@PdeNiet
Feb 23 2017 15:25
thanks!
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:25
Zrzut ekranu 2017-02-23 o 16.24.48.png
what status are you interested in ?
maybe we can add it somehow to the mainstream repo
PdeNiet
@PdeNiet
Feb 23 2017 15:26
well i've written a python script which (through conjob) asks the ralph api for all support contracts expiration date
if one is about to expire or has expired i send a mail
but some contract we want to register but not want to trigger an email, so i want to create a 'active' and 'inative' status
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:27
oh, i forgot you meant contracts object :D sorry - have to update the links
seems like a good idea to have inactive status
PdeNiet
@PdeNiet
Feb 23 2017 15:30
thanks for the pointers, i was really stuck there ^^
i see that we have only new status which is really limiting
Zrzut ekranu 2017-02-23 o 16.30.30.png
PdeNiet
@PdeNiet
Feb 23 2017 15:30
yeah thats why i wanted to use that entry
not really surving any purpose like this ;)
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:31
we can add new entry called inactive
can you do it ? and do pull request?
PdeNiet
@PdeNiet
Feb 23 2017 15:32
i'm a network engineer so i'll need to read up on programming in git ;) i can change it now locally and read up about git pull requests monday
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:33
basically -git clone https://github.com/YOURNICK/ralph
change your file and commit with git commit -m „Add new SupportStatus” -a
before this, make a fork of our repo
Zrzut ekranu 2017-02-23 o 16.34.59.png
PdeNiet
@PdeNiet
Feb 23 2017 15:38
should i commit the change to my fork then?
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:38
yes, commit to your fork
PdeNiet
@PdeNiet
Feb 23 2017 15:41
how do i keep my fork up to date with you guys on the other docs?
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:42
you can add our repo as a second remote with the:
git remote add upstream https://github.com/allegro/ralph.git
then, you can update with git pull upstream master
after the commit, remember to push to your fork with git push origin [branch] for example git push origin ng
origin = your fork
PdeNiet
@PdeNiet
Feb 23 2017 15:45
ok thanks :) need to read up on git haha
✪ vi4m Marcin Kliks
@vi4m
Feb 23 2017 15:45
:clap:
PdeNiet
@PdeNiet
Feb 23 2017 16:33
@vi4m think i did a whitespace too many, see the git pull build failed