AAROC / discourse-sso

If you're going to deploy discourse, best you deploy it with federated login
Apache License 2.0
2 stars 2 forks source link

Don't test launch #7

Closed brucellino closed 6 years ago

brucellino commented 6 years ago

Ok, this is going to be an unpopular opinion, but the launch of discourse (with ./launcher) is taking too long to test efficiently from scratch. This makes it hard to keep developing and running continuous integration tests.

There is essentially only one task which does the rebuild of the app, after all of the config is done in tasks/launch.yml so I propose to remove add an option to remove the task from the default openstack testing.

We can create a different scenario where the launch is actually tested, and this can be run at some final point in the pipeline.

brucellino commented 6 years ago

Closed in 4633bdf