Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Mar 27 18:01
    ovh-cds commented #5088
  • Mar 27 18:01
    ovh-cds commented #5088
  • Mar 27 17:41
    richardlt review_requested #5088
  • Mar 27 17:41
    richardlt opened #5088
  • Mar 27 17:41
    richardlt review_requested #5088
  • Mar 27 17:41
    richardlt review_requested #5088
  • Mar 27 17:41

    richardlt on fix-template-init

    fix(api): missing json tag to i… (compare)

  • Mar 27 14:12
    richardlt commented #5083
  • Mar 27 14:12
    richardlt commented #5083
  • Mar 27 14:11
    richardlt commented #5087
  • Mar 27 14:11
    richardlt commented #5087
  • Mar 27 14:06

    fsamin on api

    refactor(api): last crypto stuf… fix(api): unit tests fix(api): workermodel encryption and 5 more (compare)

  • Mar 27 13:23
    richardlt synchronize #5087
  • Mar 27 13:23

    richardlt on doc-template-ascode

    docs: template and workflow asc… (compare)

  • Mar 27 13:21
    sguiheux synchronize #5086
  • Mar 27 13:21

    sguiheux on workflowRunState

    fix(api): do not send pr comme… fix(api): interpolate notif eve… fix: allow run condition on joi… and 11 more (compare)

  • Mar 27 12:32
    ovh-cds commented #5087
  • Mar 27 12:32
    ovh-cds commented #5087
  • Mar 27 12:03
    richardlt review_requested #5087
  • Mar 27 12:03
    richardlt review_requested #5087
Yvonnick Esnault
@yesnault
Hi @louis-gounot , no reason. Issue created: ovh/cds#4775 About quote, you've got an error with that?
Louis GOUNOT
@louis-gounot
Yes, I wasn't able to build with extraneous "" whereas it works well without
By the way, don't understand what it would be in sedcommand
Yvonnick Esnault
@yesnault
@louis-gounot hum, it's a sed syntax for osx, issue created: ovh/cds#4778
Louis GOUNOT
@louis-gounot
@yesnault : I'm on Linux on my side, so won't be able to test it
@yesnault should work with gnu-sed (brew install gnu-sed)
Louis GOUNOT
@louis-gounot
test "${OSTYPE}" == "darwin" && SED='sed -i ""' || SED='sed -i'should do the trick also
Louis GOUNOT
@louis-gounot
@yesnault Just issued a PR for #4778, if you can confirm it works on OSX
Yvonnick Esnault
@yesnault
@louis-gounot thank you! we will check that
Yvonnick Esnault
@yesnault

CDS New Release 0.43.1

https://github.com/ovh/cds/releases/tag/0.43.1
This version is the latest version before the new authentication system available in the next release.

Happy new year!

Stijn Tintel
@stintel
Hi there! I've been using CDS for some time, on ppc64el, which is not supported. Would you consider accepting a PR that adds support for this architecture?
Yvonnick Esnault
@yesnault
Hi @stintel It's probably the first time that a user ask us to add ppc64 :) You can create an issue, we will consider it if it's possible to compile this arch. The next release won't be available before some weeks, as there is the new authentication system (oauth2) on master now, but you can try run make build OS="linux" ARCH="ppc64" (on root or in the packages engine and contrib if you don't want to compile everything) on master branch
Stijn Tintel
@stintel
@yesnault well I got my hands on a 2nd hand power8 machine with 10c/80t and 128GB RAM, I've been running all CDS components including workers on it, and I am using it to autobuild OpenWrt images for all my OpenWrt devices. I had to make some changes to be able to build it for ppc64el. would be cool if I don't have to maintain these changes. I don't recall if I tried building ppc64el binaries on an amd64 host, I'll try that and add this info to the ticket. I can also submit a PR with the changes I have, and address any remarks you have on it ...
but most of all good to know you would consider it, thanks!
Yvonnick Esnault
@yesnault
I'd just make a build with ARCH=ppc64, it's ok without changes. What kind of changes did you have to do ? It's perhaps already fixed on master branch :)
Stijn Tintel
@stintel
stijn@power8 ~/build/cds $ make build OS="linux" ARCH="ppc64"
Building CDS Components for linux - ppc64
#make build -C ui
make build -C engine OS="linux" ARCH="ppc64"
make[1]: Entering directory '/home/staff/stijn/build/cds/engine'
make lint -C sql
make[2]: Entering directory '/home/staff/stijn/build/cds/engine/sql'
make[2]: Leaving directory '/home/staff/stijn/build/cds/engine/sql'
make build -C sql
make[2]: Entering directory '/home/staff/stijn/build/cds/engine/sql'
make[2]: Leaving directory '/home/staff/stijn/build/cds/engine/sql'
Compiling ./dist/cds-engine-linux-ppc64 snapshot
make build -C worker
make[2]: Entering directory '/home/staff/stijn/build/cds/engine/worker'
create ./dist directory
Compiling ./dist/cds-worker-linux-ppc64 snapshot
make[2]: Leaving directory '/home/staff/stijn/build/cds/engine/worker'
make[1]: Leaving directory '/home/staff/stijn/build/cds/engine'
make build -C engine/worker OS="linux" ARCH="ppc64"
make[1]: Entering directory '/home/staff/stijn/build/cds/engine/worker'
Compiling ./dist/cds-worker-linux-ppc64 snapshot
make[1]: Leaving directory '/home/staff/stijn/build/cds/engine/worker'
make build -C cli/cdsctl OS="linux/%" ARCH="%/ppc64"
make[1]: Entering directory '/home/staff/stijn/build/cds/cli/cdsctl'
create ./dist directory
*** building binary dist/cdsctl-linux-ppc64
... OS:linux ARCH:ppc64 -> dist/cdsctl-linux-ppc64
# github.com/ovh/cds/cli/cdsctl/internal
internal/cdsctx.go:38:9: undefined: keychainEnabled
internal/cdsctx.go:128:12: undefined: storeTokens
internal/cdsctx.go:149:7: undefined: keychainEnabled
internal/cdsctx.go:233:5: undefined: keychainEnabled
internal/cdsctx.go:234:28: cdsContext.getTokens undefined (type CDSContext has no field or method getTokens)
make[2]: *** [Makefile:70: gobuild] Error 2
make[1]: *** [Makefile:62: dist/cdsctl-linux-ppc64] Error 2
make[1]: Leaving directory '/home/staff/stijn/build/cds/cli/cdsctl'
make: *** [Makefile:55: build] Error 2
commented out $(MAKE) build -C ui as I don't have npm installed
Yvonnick Esnault
@yesnault
with cds.manual = true in run condition, user have a modal box to enter pipeline parameter before running a pipeline
@stintel ok thank you, PR to fix that is now here: ovh/cds#4869
Stijn Tintel
@stintel
@yesnault that seems to fix building for ARCH=ppc64. with ARCH=ppc64el I get another error
many cmd/go: unsupported GOOS/GOARCH pair linux/ppc64el ... too long to post in full here
oops, it should be ARCH=ppc64le
Stijn Tintel
@stintel
ok, my bad. with your change it builds find for both ARCH=ppc64 and ARCH=ppc64le
Yvonnick Esnault
@yesnault
@stintel which arch do you use? ppc64 or ppc64le?
Stijn Tintel
@stintel
@yesnault I'm using ppc64le
Yvonnick Esnault
@yesnault

New Release - 0.44.0

@/all
It's now possible to authenticate users with oauth2 GitHub & GitLab, and of course with Ldap.

https://github.com/ovh/cds/releases/tag/0.44.0

SignIn.png
Louis GOUNOT
@louis-gounot
Hello,
Do you have any plan to move from Native Doc Types (_type) in ElasticSearch to applicative (~ field) type ?
Right now, I have to deploy ElasticSearch in 5.x then upgrade to 6.x in order to have it work.
See https://www.elastic.co/guide/en/elasticsearch/reference/current/removal-of-types.html
Related to #4528
Louis GOUNOT
@louis-gounot
Hello,
I can't deploy a brand new CDS with last release.
µServices can't register/authenticate with API.
Is there any specific setup step needed now ?
Louis GOUNOT
@louis-gounot
Hello, forgot the INIT_TOKEN stuff, signup a first user with INIT_TOKEN exported solved the issue
Yvonnick Esnault
@yesnault
Hi @louis-gounot about #4528, we'll do that for the next release
Louis GOUNOT
@louis-gounot
Thanks :-)
Louis GOUNOT
@louis-gounot
Hello,
I have some issues with Kubernetes hatchery in 0.44.0
Pods created to register worker models are created as a concatenation of register, HATCHERY_NAME, MODEL_GROUP_NAME, MODEL_NAME and CUSTOM_STRING
It causes trouble to K8S when :
  1. Generated name is too long (more than 63 characters)
  2. Name contains .(dot) character either from MODEL_GROUP_NAME => shared.infra or MODEL_NAME => golang-1.13
  3. Name contains _ (underscore) or (space) (usually coming from HATCHERY_NAME as defined in CDS (autogenerated) config file)
Is it something known ?
Yvonnick Esnault
@yesnault
Hi @louis-gounot , yes, it's should be checked here: https://github.com/ovh/cds/blob/master/engine/hatchery/kubernetes/kubernetes.go#L256, but if you have an issue about that, it's probably a new issue
Yvonnick Esnault
@yesnault
it will be fixed in the next release too
Should be podNameinstead of spawnArgs.WorkerName which basically makes podNamevar useless
Don't know if there is side effect
Louis GOUNOT
@louis-gounot
Any idea about next release date ?
Yvonnick Esnault
@yesnault
@louis-gounot fix is on ovh/cds#5013, release planned for tomorrow for now
Louis GOUNOT
@louis-gounot
:smile:
Yvonnick Esnault
@yesnault

New Release 0.45.0

https://github.com/ovh/cds/releases/tag/0.45.0

cc @/all @louis-gounot

Louis GOUNOT
@louis-gounot
Is #5005 included ? I don't see it in changelog
Yvonnick Esnault
@yesnault
@louis-gounot it's in changelog, with title 'this: engine: gitlab user auth with custom endpoint (#5006) (f30e8f0)', commit ovh/cds@f30e8f0
but, the (generated changelog) say that it close the 5008 instead of 5005
Louis GOUNOT
@louis-gounot
OK, :thumbsup: