V2.0-beta #50

DavyJohnes · 2021-12-27T13:07:09Z

No description provided.

BREAKING CHANGE: changed API of service clients

BREAKING CHANGE: changed API of Session

BREAKING CHANGE: removed some useless classes

Test semmantic release

# [2.0.0-alpha.1](v1.4.3...v2.0.0-alpha.1) (2021-12-24) ### Bug Fixes * cache node_modules between jobs ([0a9490a](0a9490a)) * disable husky hooks in CI ([4919d54](4919d54)) * eslint rule ([e097134](e097134)) * move legacy code to separate directory ([b38248c](b38248c)) * move to axios ([438f7c6](438f7c6)) * remove prettier ([6bf0f24](6bf0f24)) * remove unused test module ([4e90a92](4e90a92)) * update ts-proto ([a20a650](a20a650)) * use bot's name/email for pushing release commits ([3b2553a](3b2553a)) * use bot's token for pushing release commits ([76cf6e7](76cf6e7)) ### Features * added git submodule with yandex cloud api specs ([7916612](7916612)) * endpoints list for all available service clients ([78c2355](78c2355)) * generate exports for all entities in cloud api ([5136bb5](5136bb5)) * operation utilities ([e82e279](e82e279)) * regenerate code with new ts-proto ([1e2dcc2](1e2dcc2)) * remove old generated code ([a5ecb33](a5ecb33)) * remove unused legacy code and dependencies ([3d3a6f2](3d3a6f2)) * restructure directories ([55bf685](55bf685)) * reworked examples ([2875275](2875275)) * rewrite index module in ts ([99e8ba8](99e8ba8)) * session class ([826e6de](826e6de)) * token services ([8b730df](8b730df)) * tool for code generation ([b463cf8](b463cf8)) ### BREAKING CHANGES * removed some useless classes * changed API of Session * changed API of service clients

(cherry picked from commit ad72226)

DavyJohnes · 2022-01-12T11:44:59Z

@DavyJohnes Кажется тут ошибка. https://github.com/yandex-cloud/nodejs-sdk/blob/alpha/src/service-endpoints.ts#L164-L169
{
        serviceIds: [
            'yandex.cloud.logging.v1.LogGroupService',
            'yandex.cloud.logging.v1.LogIngestionService',
            'yandex.cloud.logging.v1.LogReadingService',
        ],
        endpoint: 'logs.api.cloud.yandex.net:443',
    },
А должно быть как тут https://api.cloud.yandex.net/endpoints ingester.logging.yandexcloud.net:443

Спасибо! Пофикшено в 2.0.0-alpha.3. Более того, в session.client() добавил второй параметр, чтобы иметь возможность переопределить эндпоинт.

@pltnm239 попробуете еще раз?

pltnm239 · 2022-01-12T12:13:11Z

@DavyJohnes Кажется тут ошибка. https://github.com/yandex-cloud/nodejs-sdk/blob/alpha/src/service-endpoints.ts#L164-L169
{
        serviceIds: [
            'yandex.cloud.logging.v1.LogGroupService',
            'yandex.cloud.logging.v1.LogIngestionService',
            'yandex.cloud.logging.v1.LogReadingService',
        ],
        endpoint: 'logs.api.cloud.yandex.net:443',
    },
А должно быть как тут https://api.cloud.yandex.net/endpoints ingester.logging.yandexcloud.net:443
Спасибо! Пофикшено в 2.0.0-alpha.3. Более того, в session.client() добавил второй параметр, чтобы иметь возможность переопределить эндпоинт.

@pltnm239 попробуете еще раз?

Пробую, теперь ошибка: UnhandledPromiseRejectionWarning: ClientError: /yandex.cloud.logging.v1.LogIngestionService/Write INVALID_ARGUMENT: no valid entries хотя entries по тайпингам валидный.

DavyJohnes · 2022-01-12T12:20:07Z

@DavyJohnes Кажется тут ошибка. https://github.com/yandex-cloud/nodejs-sdk/blob/alpha/src/service-endpoints.ts#L164-L169
{
        serviceIds: [
            'yandex.cloud.logging.v1.LogGroupService',
            'yandex.cloud.logging.v1.LogIngestionService',
            'yandex.cloud.logging.v1.LogReadingService',
        ],
        endpoint: 'logs.api.cloud.yandex.net:443',
    },
А должно быть как тут https://api.cloud.yandex.net/endpoints ingester.logging.yandexcloud.net:443
Спасибо! Пофикшено в 2.0.0-alpha.3. Более того, в session.client() добавил второй параметр, чтобы иметь возможность переопределить эндпоинт.
@pltnm239 попробуете еще раз?
Пробую, теперь ошибка: UnhandledPromiseRejectionWarning: ClientError: /yandex.cloud.logging.v1.LogIngestionService/Write INVALID_ARGUMENT: no valid entries хотя entries по тайпингам валидный.

Подозреваю там level является обязательным полем. По поводу тайпингов замечание справедливое, подумаю что с этим сделать.

export interface IncomingLogEntry {
  $type: "yandex.cloud.logging.v1.IncomingLogEntry";
  /** Timestamp of the entry. */
  timestamp?: Date;
  /**
   * Entry severity.
   *
   * See [LogLevel.Level] for details.
   */
  level: LogLevel_Level;
  /** Entry text message. */
  message: string;
  /** Entry annotation. */
  jsonPayload?: { [key: string]: any };
}

pltnm239 · 2022-01-12T12:27:02Z

@DavyJohnes Кажется тут ошибка. https://github.com/yandex-cloud/nodejs-sdk/blob/alpha/src/service-endpoints.ts#L164-L169
{
        serviceIds: [
            'yandex.cloud.logging.v1.LogGroupService',
            'yandex.cloud.logging.v1.LogIngestionService',
            'yandex.cloud.logging.v1.LogReadingService',
        ],
        endpoint: 'logs.api.cloud.yandex.net:443',
    },
А должно быть как тут https://api.cloud.yandex.net/endpoints ingester.logging.yandexcloud.net:443
Спасибо! Пофикшено в 2.0.0-alpha.3. Более того, в session.client() добавил второй параметр, чтобы иметь возможность переопределить эндпоинт.
@pltnm239 попробуете еще раз?
Пробую, теперь ошибка: UnhandledPromiseRejectionWarning: ClientError: /yandex.cloud.logging.v1.LogIngestionService/Write INVALID_ARGUMENT: no valid entries хотя entries по тайпингам валидный.
Подозреваю там level является обязательным полем. По поводу тайпингов замечание справедливое, подумаю что с этим сделать.
export interface IncomingLogEntry {
  $type: "yandex.cloud.logging.v1.IncomingLogEntry";
  /** Timestamp of the entry. */
  timestamp?: Date;
  /**
   * Entry severity.
   *
   * See [LogLevel.Level] for details.
   */
  level: LogLevel_Level;
  /** Entry text message. */
  message: string;
  /** Entry annotation. */
  jsonPayload?: { [key: string]: any };
}

А я не могу ориентироваться на документацию https://cloud.yandex.ru/docs/logging/api-ref/grpc/log_ingestion_service ?
Тут из обязательных только destination.

Странно, но указание level не помогает (ни числовое ни строковое значение из списка). Ошибка та же самая.

DavyJohnes · 2022-01-12T12:32:55Z

А я не могу ориентироваться на документацию https://cloud.yandex.ru/docs/logging/api-ref/grpc/log_ingestion_service ? Тут из обязательных только destination.

Странно, но указание level не помогает (ни числовое ни строковое значение из списка). Ошибка та же самая.

Да, можно ориентироваться на документацию. Чуть позже гляну в чем там дело.

nikolaymatrosov · 2022-01-12T14:27:25Z

@pltnm239 поле entries[].timestamp обязательно. Без него записи не проходят валидацию, отбрасываются и вы собственно получаете ошибку, что валидных записей в запросе не передано: no valid entries.

pltnm239 · 2022-01-13T12:13:01Z

Может вы поможете разобраться, в чем же дело:)

Если комментирую TypeError тут node_modules/@grpc/grpc-js/build/src/channel.js, и оно работает, логи отправляются.

        if (!(credentials instanceof channel_credentials_1.ChannelCredentials)) {
			console.log(credentials);
			console.log(channel_credentials_1.ChannelCredentials);
            // throw new TypeError('Channel credentials must be a ChannelCredentials object');
        }

В консоли два объекта:

ComposedChannelCredentialsImpl {
  callCredentials: SingleCallCredentials { metadataGenerator: [Function (anonymous)] },
  channelCredentials: SecureChannelCredentialsImpl {
    callCredentials: EmptyCallCredentials {},
    rootCerts: null,
    privateKey: null,
    certChain: null,
    verifyOptions: {},
    connectionOptions: { secureContext: [SecureContext] }
  }
}
[class ChannelCredentials]
SecureChannelCredentialsImpl {
  callCredentials: EmptyCallCredentials {},
  rootCerts: null,
  privateKey: null,
  certChain: null,
  verifyOptions: {},
  connectionOptions: { secureContext: SecureContext { context: SecureContext {} } }
}
[class ChannelCredentials]

По удалял по одной все зависимости, пытаясь найти где же может быть коллизия. Но это не помогло:(

Fix missing "long" package

# [2.0.0-alpha.4](v2.0.0-alpha.3...v2.0.0-alpha.4) (2022-01-13) ### Bug Fixes * install long package ([7120858](7120858))

Rename npm package

Change package description

# [2.0.0-alpha.5](v2.0.0-alpha.4...v2.0.0-alpha.5) (2022-01-20) ### Features * change package description ([e9ecf16](e9ecf16))

Update package description

# [2.0.0-alpha.6](v2.0.0-alpha.5...v2.0.0-alpha.6) (2022-01-20) ### Bug Fixes * set public access for npm package ([a68360d](a68360d))

Use exact versions for dependencies

# [2.0.0-alpha.7](v2.0.0-alpha.6...v2.0.0-alpha.7) (2022-01-20) ### Bug Fixes * use exact versions of dependencies ([b564178](b564178))

Update examples

yandex-cloud-bot · 2022-01-20T16:18:19Z

🎉 This PR is included in version 2.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

yandex-cloud-bot · 2022-04-01T11:23:41Z

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Ivan Zuev and others added 30 commits December 15, 2021 14:51

fix: remove unused test module

4e90a92

feat: restructure directories

55bf685

feat: rewrite index module in ts

99e8ba8

feat: added git submodule with yandex cloud api specs

7916612

feat: remove old generated code

a5ecb33

fix: eslint rule

e097134

feat: tool for code generation

b463cf8

feat: endpoints list for all available service clients

78c2355

fix: move legacy code to separate directory

b38248c

feat: token services

8b730df

feat: regenerate code with new ts-proto

1e2dcc2

BREAKING CHANGE: changed API of service clients

feat: session class

826e6de

BREAKING CHANGE: changed API of Session

feat: operation utilities

e82e279

test: fix unit tests

9cbffe1

feat: remove unused legacy code and dependencies

3d3a6f2

BREAKING CHANGE: removed some useless classes

fix: move to axios

438f7c6

feat: generate exports for all entities in cloud api

5136bb5

feat: reworked examples

2875275

fix: update ts-proto

a20a650

style: conventional commits as pre-commit hooks

718d836

ci: semantic-release

549c9aa

fix: disable husky hooks in CI

4919d54

fix: use bot's token for pushing release commits

76cf6e7

fix: use bot's name/email for pushing release commits

3b2553a

ci: option to override persist-credentials in checkout action

90be359

ci: do not persist credential during checkout

8cae8f3

Merge pull request #48 from yandex-cloud/CLOUD-86841

0dfcbba

Test semmantic release

ci: do not run build on npm install

ad72226

ci: do not run build on npm install

e9edd44

(cherry picked from commit ad72226)

DavyJohnes mentioned this pull request Jan 13, 2022

TypeError in node_modules/@grpc/grpc-js/build/src/channel.js #56

Closed

Ivan Zuev and others added 17 commits January 13, 2022 16:09

fix: install long package

7120858

Merge pull request #57 from yandex-cloud/CLOUD-86841

0ef6ca8

Fix missing "long" package

chore(release): 2.0.0-alpha.4 [skip ci]

6e2c64b

# [2.0.0-alpha.4](v2.0.0-alpha.3...v2.0.0-alpha.4) (2022-01-13) ### Bug Fixes * install long package ([7120858](7120858))

refactor: rename npm package to @yandex-cloud/nodejs-sdk

795d541

Merge pull request #58 from yandex-cloud/CLOUD-86841

1a90dc6

Rename npm package

feat: change package description

e9ecf16

Merge pull request #59 from yandex-cloud/CLOUD-86841

9a90298

Change package description

chore(release): 2.0.0-alpha.5 [skip ci]

9deb1db

# [2.0.0-alpha.5](v2.0.0-alpha.4...v2.0.0-alpha.5) (2022-01-20) ### Features * change package description ([e9ecf16](e9ecf16))

fix: set public access for npm package

a68360d

Merge pull request #60 from yandex-cloud/CLOUD-86841

ed5afb6

Update package description

chore(release): 2.0.0-alpha.6 [skip ci]

a30fcec

# [2.0.0-alpha.6](v2.0.0-alpha.5...v2.0.0-alpha.6) (2022-01-20) ### Bug Fixes * set public access for npm package ([a68360d](a68360d))

refactor: update readme

6a462d9

fix: use exact versions of dependencies

b564178

Merge pull request #61 from yandex-cloud/CLOUD-86841

ef3857e

Use exact versions for dependencies

chore(release): 2.0.0-alpha.7 [skip ci]

a5ecdb7

# [2.0.0-alpha.7](v2.0.0-alpha.6...v2.0.0-alpha.7) (2022-01-20) ### Bug Fixes * use exact versions of dependencies ([b564178](b564178))

docs: update examples

29fc31f

Merge pull request #62 from yandex-cloud/CLOUD-86841

280ce25

Update examples

DavyJohnes merged commit 8dc3835 into beta Jan 20, 2022

yandex-cloud-bot added the released on @beta label Jan 20, 2022

yandex-cloud-bot added the released label Apr 1, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

V2.0-beta #50

V2.0-beta #50

DavyJohnes commented Dec 27, 2021

DavyJohnes commented Jan 12, 2022

pltnm239 commented Jan 12, 2022

DavyJohnes commented Jan 12, 2022 •

edited

Loading

pltnm239 commented Jan 12, 2022 •

edited

Loading

DavyJohnes commented Jan 12, 2022

nikolaymatrosov commented Jan 12, 2022

pltnm239 commented Jan 13, 2022 •

edited

Loading

yandex-cloud-bot commented Jan 20, 2022

yandex-cloud-bot commented Apr 1, 2022

V2.0-beta #50

V2.0-beta #50

Conversation

DavyJohnes commented Dec 27, 2021

DavyJohnes commented Jan 12, 2022

pltnm239 commented Jan 12, 2022

DavyJohnes commented Jan 12, 2022 • edited Loading

pltnm239 commented Jan 12, 2022 • edited Loading

DavyJohnes commented Jan 12, 2022

nikolaymatrosov commented Jan 12, 2022

pltnm239 commented Jan 13, 2022 • edited Loading

yandex-cloud-bot commented Jan 20, 2022

yandex-cloud-bot commented Apr 1, 2022

DavyJohnes commented Jan 12, 2022 •

edited

Loading

pltnm239 commented Jan 12, 2022 •

edited

Loading

pltnm239 commented Jan 13, 2022 •

edited

Loading