Skip to content
This repository was archived by the owner on Oct 3, 2023. It is now read-only.

Add Example applications and documentation#74

Merged
chingor13 merged 16 commits intomasterfrom
example-apps
Nov 28, 2017
Merged

Add Example applications and documentation#74
chingor13 merged 16 commits intomasterfrom
example-apps

Conversation

@chingor13
Copy link
Copy Markdown
Member

  • Adds a few skeleton applications: Laravel 5.5, Symfony 3.3, Silex 2.2. Each application has a README with a description of how to enable OpenCensus tracing.

Fixes #61

@chingor13 chingor13 requested a review from tmatsuo November 22, 2017 18:01
@chingor13 chingor13 changed the title WIP: Add Example applications and documentation Add Example applications and documentation Nov 22, 2017
Copy link
Copy Markdown
Contributor

@tmatsuo tmatsuo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm cool to include the skeleton code, but I'm also wondering if there's a clean way of generating the skeleton code from the source somehow.

Comment thread examples/symfony/composer.json Outdated
"symfony/swiftmailer-bundle": "^2.3.10",
"symfony/symfony": "3.3.*",
"twig/twig": "^1.0||^2.0",
"opencensus/opencensus": "dev-example-apps",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

Comment thread examples/laravel/composer.json Outdated
"fideloper/proxy": "~3.3",
"laravel/framework": "5.5.*",
"laravel/tinker": "~1.0",
"opencensus/opencensus": "dev-example-apps",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be tricky, but I think we eventually want to change it from "dev-example-apps"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I needed to set it to something. I'll switch them to dev-master for now.

Comment thread examples/silex/composer.json Outdated
"type": "project",
"require": {
"silex/silex": "^2.2",
"opencensus/opencensus": "dev-example-apps",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto

@@ -0,0 +1,19 @@
<?php

require_once __DIR__ . '/../vendor/autoload.php';
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this app trace?

How about to add an example for tracing the web handlers?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A root span is created for the main web request which is named after the requested path.

I'll add another inline example within one for the request handlers.


namespace AppBundle;

use OpenCensus\Trace\Exporter\StackdriverExporter;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you need these imports?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope, I'll remove. These were remnants from when I had the integrations directly in AppBundle instead of the OpenCensusBundle

@chingor13 chingor13 merged commit d6398bb into master Nov 28, 2017
@chingor13 chingor13 deleted the example-apps branch November 28, 2017 21:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants