Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@chemelnucfin
Copy link
Contributor

@chemelnucfin chemelnucfin commented Jan 28, 2018

Towards #4692

Jon Wayne Parrott added 3 commits January 26, 2018 12:44
Command log:

```
set DEST ~/workspace/google-cloud-python/trace
set GOOGLEAPIS_REV origin/master

cd /tmp
git clone https://github.com/googleapis/googleapis.git
cd googleapis
git reset --hard $GOOGLEAPIS_REV
artman --config google/devtools/cloudtrace/artman_cloudtrace_v1.yaml generate python_gapic
artman --config google/devtools/cloudtrace/artman_cloudtrace_v2.yaml generate python_gapic

set SRC ./artman-genfiles/python/trace-v1/
set IMPORT_PKG trace_v1
cp -r $SRC/docs $DEST
cp -r $SRC/google/cloud/$IMPORT_PKG $DEST/google/cloud/
mkdir -p $DEST/tests/unit/gapic
cp -r $SRC/tests/unit/gapic $DEST/tests/unit
mkdir -p $DEST/tests/system/gapic
cp -r $SRC/tests/system/gapic $DEST/tests/system

set SRC ./artman-genfiles/python/trace-v2/
set IMPORT_PKG trace_v2
cp -r $SRC/docs $DEST
cp -r $SRC/google/cloud/$IMPORT_PKG $DEST/google/cloud/
mkdir -p $DEST/tests/unit/gapic
cp -r $SRC/tests/unit/gapic $DEST/tests/unit
mkdir -p $DEST/tests/system/gapic
cp -r $SRC/tests/system/gapic $DEST/tests/system
```
@chemelnucfin chemelnucfin added api: cloudtrace Issues related to the Cloud Trace API. type: cleanup An internal cleanup or hygiene concern. labels Jan 28, 2018
@chemelnucfin chemelnucfin self-assigned this Jan 28, 2018
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 28, 2018
@chemelnucfin chemelnucfin changed the title Regen trace Fix trace tests after gax to gapic Jan 28, 2018
@chemelnucfin chemelnucfin changed the title Fix trace tests after gax to gapic Trace: Fix unittests after gax to gapic Jan 28, 2018
@theacodes theacodes changed the title Trace: Fix unittests after gax to gapic Regenerate gapic for Trace and remove dependency on gax Feb 2, 2018
@theacodes
Copy link
Contributor

Thanks @chemelnucfin! Can you draft a release of this?

@chemelnucfin
Copy link
Contributor Author

@jonparrott Merge it first?

@theacodes
Copy link
Contributor

yep

@chemelnucfin chemelnucfin merged commit b2e7953 into googleapis:master Feb 2, 2018
@chemelnucfin chemelnucfin deleted the regen-trace branch February 2, 2018 06:29
parthea pushed a commit that referenced this pull request Oct 22, 2023
* Import trace_v1 and trace_v2 generated by artman

Command log:

```
set DEST ~/workspace/google-cloud-python/trace
set GOOGLEAPIS_REV origin/master

cd /tmp
git clone https://github.com/googleapis/googleapis.git
cd googleapis
git reset --hard $GOOGLEAPIS_REV
artman --config google/devtools/cloudtrace/artman_cloudtrace_v1.yaml generate python_gapic
artman --config google/devtools/cloudtrace/artman_cloudtrace_v2.yaml generate python_gapic

set SRC ./artman-genfiles/python/trace-v1/
set IMPORT_PKG trace_v1
cp -r $SRC/docs $DEST
cp -r $SRC/google/cloud/$IMPORT_PKG $DEST/google/cloud/
mkdir -p $DEST/tests/unit/gapic
cp -r $SRC/tests/unit/gapic $DEST/tests/unit
mkdir -p $DEST/tests/system/gapic
cp -r $SRC/tests/system/gapic $DEST/tests/system

set SRC ./artman-genfiles/python/trace-v2/
set IMPORT_PKG trace_v2
cp -r $SRC/docs $DEST
cp -r $SRC/google/cloud/$IMPORT_PKG $DEST/google/cloud/
mkdir -p $DEST/tests/unit/gapic
cp -r $SRC/tests/unit/gapic $DEST/tests/unit
mkdir -p $DEST/tests/system/gapic
cp -r $SRC/tests/system/gapic $DEST/tests/system
```

* Fix bug in autogen: invalid import paths in proto files

* Remove references to gax

* Trace: Fix tests after replacing gax with gapic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: cloudtrace Issues related to the Cloud Trace API. cla: yes This human has signed the Contributor License Agreement. type: cleanup An internal cleanup or hygiene concern.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants