Exposing apps from kubernetes cluster

Hello,
Successfully created all the resources by the help of this repo.
Also installed jenkins container inside operator host using helm install.

I want to access it in the internet.
What are the possible configurations to do?
Thank you so much.

terrafirn.tfvars
# Copyright 2017, 2021 Oracle Corporation and/or affiliates.
# Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl

# Identity and access parameters
#api_fingerprint      = "${OCI_API_FINGERPRINT}"
# api_private_key      = <<EOT
#-----BEGIN RSA PRIVATE KEY-----
#content+of+api+key
#-----END RSA PRIVATE KEY-----
#EOT

api_private_key_path = "./key.pem"

home_region = "us-sanjose-1"
region = "us-sanjose-1"

#tenancy_id           = "${OCI_TENANCY_ID}"
#user_id              = "${OCI_USER_ID}"

# general oci parameters
#compartment_id = "${OCI_COMPARTNMENT_PAT_TEST}"
#label_prefix   = "${OCI_COMPARTNMENT_PAT_TEST_PREFIX}"  # (test) sub-compartment of pat.ai

# ssh keys
#ssh_private_key      = "${OCI_SSH_PRIVATE_KEY}"
# ssh_private_key    = <<EOT
#-----BEGIN RSA PRIVATE KEY-----
#content+of+api+key
#-----END RSA PRIVATE KEY-----
#EOT
ssh_private_key_path = "./ssh.key"
# ssh_public_key       = "${OCI_SSH_PUBLIC_KEY}"
# ssh_public_key_path  = "none"
ssh_public_key_path = "./ssh.pub"

# networking
create_drg                   = false
drg_display_name             = "drg"

internet_gateway_route_rules = []

local_peering_gateways = {}

lockdown_default_seclist = true

nat_gateway_route_rules = []

nat_gateway_public_ip_id = "none"

subnets = {
  bastion  = { netnum = 0, newbits = 13 }
  operator = { netnum = 1, newbits = 13 }
  cp       = { netnum = 2, newbits = 13 }
  int_lb   = { netnum = 16, newbits = 11 }
  pub_lb   = { netnum = 17, newbits = 11 }
  workers  = { netnum = 1, newbits = 2 }
  fss      = { netnum = 18, newbits = 11 }
}

vcn_cidrs     = ["<IP_ADDRESS>/16"]
vcn_dns_label = "oke"
vcn_name      = "vcnoke"

# bastion host
create_bastion_host = true
bastion_access      = ["anywhere"]
bastion_image_id    = "Autonomous"
bastion_os_version  = "7.9"
bastion_shape = {
  shape            = "VM.Standard.E3.Flex",
  ocpus            = 1,
  memory           = 4,
  boot_volume_size = 50
}
bastion_state    = "RUNNING"
bastion_timezone = "Etc/UTC"
bastion_type     = "public"
upgrade_bastion  = false

## bastion notification
enable_bastion_notification   = false
bastion_notification_endpoint = ""
bastion_notification_protocol = "EMAIL"
bastion_notification_topic    = "bastion_server_notification"

# bastion service
create_bastion_service        = false
bastion_service_access        = ["<IP_ADDRESS>/0"]
bastion_service_name          = "bastion"
bastion_service_target_subnet = "operator"

# operator host
create_operator                    = true
operator_image_id                  = "Oracle"
enable_operator_instance_principal = true
operator_nsg_ids                   = []
operator_os_version                = "8"
operator_shape = {
  shape            = "VM.Standard.E4.Flex",
  ocpus            = 1,
  memory           = 4,
  boot_volume_size = 50
}
operator_state    = "RUNNING"
operator_timezone = "Etc/UTC"
upgrade_operator  = false

# Operator in-transit encryption for the data volume's paravirtualized attachment.
enable_operator_pv_encryption_in_transit = false

# operator volume kms integration
operator_volume_kms_id = ""

## operator notification
enable_operator_notification   = false
operator_notification_endpoint = ""
operator_notification_protocol = "EMAIL"
operator_notification_topic    = ""

# availability_domains
availability_domains = {
  bastion  = 1,
  operator = 1,
  fss      = 1
}

# oke cluster options
admission_controller_options = {
  PodSecurityPolicy = false
}
allow_node_port_access       = false
allow_worker_internet_access = true
allow_worker_ssh_access      = false
cluster_name                 = "oke"
control_plane_type           = "public"
control_plane_allowed_cidrs  = ["<IP_ADDRESS>/0"]
control_plane_nsgs           = []
dashboard_enabled            = true
kubernetes_version           = "v1.21.5"
pods_cidr                    = "<IP_ADDRESS>/16"
services_cidr                = "<IP_ADDRESS>/16"

## oke cluster kms integration
use_cluster_encryption = false
#cluster_kms_key_id     = "${OCI_CLUSTER_KMS_KEY_ID}" #OKE-SOFTWARE-AES

### oke node pool volume kms integration
use_node_pool_volume_encryption = false
node_pool_volume_kms_key_id     = ""

## oke cluster container image policy and keys
use_signed_images = false
image_signing_keys = []

# node pools
check_node_active = "all"
enable_pv_encryption_in_transit = false
node_pools = {
  np1 = { shape = "VM.Standard.E4.Flex", ocpus = 1, memory = 16, node_pool_size = 1, boot_volume_size = 150, label = { app = "frontend", pool = "np1" } }
  np2 = { shape = "VM.Standard.E4.Flex", ocpus = 1, memory = 16, node_pool_size = 1, boot_volume_size = 150, label = { app = "frontend", pool = "np2" } }
  # np3 = { shape = "VM.Standard.E4.Flex", ocpus = 1, memory = 16, node_pool_size = 1, boot_volume_size = 150, label = { app = "frontend", pool = "np1" } }
  # np4 = { shape = "VM.Standard.E4.Flex", ocpus = 1, memory = 16, node_pool_size = 1, boot_volume_size = 150, label = { app = "frontend", pool = "np1" } }
  # np2 = {shape="VM.Standard.E4.Flex",ocpus=4,memory=16,node_pool_size=1,boot_volume_size=150, label={app="backend",pool="np2"}}
  # np3 = {shape="VM.Standard.A1.Flex",ocpus=8,memory=16,node_pool_size=1,boot_volume_size=150, label={pool="np3"}}
  # np4 = {shape="BM.Standard2.52",node_pool_size=1,boot_volume_size=150}
  # np5 = {shape="VM.Optimized3.Flex",node_pool_size=6}
  # np5 = {shape="BM.Standard.A1.160",node_pool_size=6}
  # np6 = {shape="VM.Standard.E2.2", node_pool_size=5}
  # np7 = {shape="BM.DenseIO2.52", node_pool_size=5}
  # np8 = {shape="BM.GPU3.8", node_pool_size=1}
  # np9 = {shape="BM.GPU4.8", node_pool_size=5}
  # np10 = {shape="BM.HPC2.36	", node_pool_size=5}
}
node_pool_image_id    = "none"
node_pool_name_prefix = "np"
node_pool_os          = "Oracle Linux"
node_pool_os_version  = "7.9"
node_pool_timezone    = "Etc/UTC"
worker_nsgs           = []
worker_type           = "private"

# upgrade of existing node pools
upgrade_nodepool        = false
node_pools_to_drain     = ["np1", "np2"]
nodepool_upgrade_method = "out_of_place"

# oke load balancers
enable_waf                   = false
load_balancers               = "both"
preferred_load_balancer      = "public"
# internal_lb_allowed_cidrs  = ["<IP_ADDRESS>/24", "<IP_ADDRESS>/24"] # By default, anywhere i.e. <IP_ADDRESS>/0 is allowed
internal_lb_allowed_ports  = [80, 443, "7001-7005", 30000, 8080] # By default, only 80 and 443 are allowed
# public_lb_allowed_cidrs    = ["<IP_ADDRESS>/0"] # By default, anywhere i.e. <IP_ADDRESS>/0 is allowed
public_lb_allowed_ports    = [443,"9001-9002", 30000, 8080] # By default, only 443 is allowed

#fss
create_fss = true
fss_mount_path = "/oke_fss"
max_fs_stat_bytes =<PHONE_NUMBER>3
max_fs_stat_files = 223442

# ocir
#email_address    = "${OCI_CONTAINER_REGISTRY_EMAIL}"
secret_id        = "none"  # ${OCI_CONTAINER_REGISTRY_SECRET_ID}
#secret_name      = "${OCI_CONTAINER_REGISTRY_SECRET_NAME}"
#secret_namespace = "${OCI_CONTAINER_REGISTRY_SECRET_NAMESPACE}"
#username         = "${OCI_CONTAINER_REGISTRY_USERNAME}"

# calico
enable_calico  = false
calico_version = "3.19"

# horizontal and vertical pod autoscaling
enable_metric_server = true
enable_vpa           = true
vpa_version          = 0.8

#OPA Gatekeeper
enable_gatekeeper = false
gatekeeeper_version = "3.7"

# service account
create_service_account               = true
service_account_name                 = "kubeconfigsa"
service_account_namespace            = "kube-system"
service_account_cluster_role_binding = "cicd"

# freeform_tags
freeform_tags = {
  # vcn, bastion and operator freeform_tags are required
  # add more freeform_tags in each as desired
  vcn = {
    environment = "dev"
  }
  bastion = {
    access      = "public",
    environment = "dev",
    role        = "bastion",
    security    = "high"
  }
  operator = {
    access      = "restricted",
    environment = "dev",
    role        = "operator",
    security    = "high"
  }
  oke = {
    service_lb  = {
      environment = "dev"
      role        = "load balancer"
    }
  }
}

# placeholder variable for debugging scripts. To be implemented in future
debug_mode = false

There are a couple of ways you can achieve this:

You can tunnel to the operator host and run kubectl port-forwarding but you'll have to do that everytime
You can deploy an Ingress Controller and create an ingress to route your requests to the Jenkins service.

Since you are exposing your Jenkins publicly, please ensure you've secured it properly.
Hope that that helps.

This is rather a problem on the way the ingress controller or LBaaS service is set rather than Kubernetes. As @hyder says, you can either deploy an ingress controller and then do a routing or publish a service based on a clustered service and then expose it through an LBaaS.
I'd suggest to close this issue as there is no relationship with OKE in here.

@hyder thank you with @dralquinta help we figured it out we were trying to access the Jenkins via the NAT Gateway instead of using the public LB as its was not clear to us in the diagram. Maybe diagram could be updated to be clear on that part.


Lua error when using configuration options

On init.vim after installing the plugin:
lua << EOF
require('github-theme').setup({
  themeStyle = "dimmed",
})
EOF

returns an error
E5108: Error executing lua ...im/plugged/github-nvim-theme/lua/github-theme/config.lua:33: attempt to
concatenate global 'vim' (a table value)

Using a minimal neovim config also returns a similar error.

For the context, this works fine
lua << EOF
require('github-theme').setup({})
EOF

Are you using nvim >= 0.5?

Yes, using neovim 0.5

Can you able to share the nvim config?

Here is the minimal nvim config:
call plug#begin('~/.vim/plugged')
Plug 'projekt0n/github-nvim-theme'
call plug#end()

lua << EOF
require('github-theme').setup({
  themeStyle = 'dimmed'
})
EOF

running nvim -v returns:
NVIM v0.5.0
Build type: Release
LuaJIT 2.1.0-beta3
Compilation: clang -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -O2 -DNDEBUG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/tmp/neovim-20210702-14987-rep9di/neovim-0.5.0/build/config -I/tmp/neovim-20210702-14987-rep9di/neovim-0.5.0/src -I/usr/local/include -I/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include -I/usr/local/opt/gettext/include -I/tmp/neovim-20210702-14987-rep9di/neovim-0.5.0/build/src/nvim/auto -I/tmp/neovim-20210702-14987-rep9di/neovim-0.5.0/build/include
Compiled by brew@BigSur

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.5.0/share/nvim"

Run :checkhealth for more info

I tried your config with nvim-nightly. It's working fine. I tweaked the way to assign global variables on my fork. Give it a try.
call plug#begin('~/.vim/plugged')
Plug 'ful1e5/github-nvim-theme', { 'branch': 'refactor/config.lua' }
call plug#end()

lua << EOF
require('github-theme').setup({
  themeStyle = 'dimmed'
})
EOF

lt changes the theme without error now, but I noticed it overwrites some configs that are declared before it.

Share your config.

lt changes the theme without error now, but I noticed it overwrites some configs that are declared before it.

That configs are related to github-nvim-theme?

I pushed 9fe4e90 on ful1e5/github-nvim-theme . give it try.

Sorry, just had time to look into this again.

I pushed 9fe4e90 on ful1e5/github-nvim-theme . give it try.
This seems to work perfectly fine now! Thanks for this! 🎉


Add changelog support and documentation

Coverage remained the same at 100.0% when pulling cc11a3fe72e59b2ced2292386291dff8e8554684 on feature/changelog into 791656e3888b2098830fcc880a9df62249a8b5f9 on breaking/v0.4.0.


docs: add JSON schema for moddle schema

This PR adds a JSON schema at moddle/docs/moddle.json which both allows validation of the existing schemas and helps in writing or extending schemas. The schema itself is validated via ajv on the existing test fixtures.

Great improvement! :)


pls add ultimate member if can pls

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Please could someone add a ultimate membership hack that you can use to it's full extent, because when I equip a mythical epic, it doesn't put it in a battle when I fight another pet.

how to unlock mythical epics


Nuget Packaging

I'm preparing to create a build with VSTS Online with CRM 365 but it does not appear that nuget package is available for reference in the NuGet Installer task. Is there a work around for this or can you put this into your build process so that it's available?

What are you looking to get packaged in Nuget? The PowerShell dlls and the scripts that I provide?
I know some users store the scripts and Dlls in source control and execute it that way.

I would think that only the .DLLs would be in the packages. Currently, I'm using your code in an unlisted nuget to try to get the XRM Framework working in a PS build task to just restore and reference the DLLs. Then in my build task i could use your scripts, use my scripts or whatever to invoke the methods in the dlls.
As for the PS scripts, my feeling is that your PS scripts are excellent scaffolding where I would guess people would take those and adjust them as needed.

When you do get this going, I did have an issue with restoring the XRM Framework from nuget and referencing the correct .DLLs. For example, I had to copy Xrm.Framework.CI.PowerShell.Cmdlets.dll nuget packages to the \bin\debug folder to get this to work. I'd be glad to send a more full email to you, just DM the tweet I sent you asking for your email :)

Sounds great. I will create a nugget package for the dll.

Hi, here is the public nugget package: https://www.nuget.org/packages/XrmCIFramework/
Would you be able to give it a try and let me know if it works for you?
I have pushed the changes too.

Sure thing, will let you know in a day or so.
Sent from my iPhone

On Apr 19, 2017, at 2:46 PM, WaelHamze<EMAIL_ADDRESS>wrote:
Hi, here is the public nugget package: https://www.nuget.org/packages/XrmCIFramework/
Would you be able to give it a try and let me know if it works for you?
I have pushed the changes too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Excellent take your time.

I will close this issue for now. Please re-open if you face any issues. Would be good to know if this works for you anyway :)

Sounds good and will do. We got killed last week with a CRM/Scribe prod issue so I wasn't able to validate the new nugget package. This week should be better as we've resolved it and will carve out time. Thanks for your work.
On a similar note but different context, would you be able to create a real world demo of CI and your tooling? I work at RSM and we've discussed at length about increasing crm capacity among crm developers but have yet to be provided a real how-to from Microsoft or Sonoma Partners. I believe it's possible but I cannot connect the "process dots", if you will. If you believe you may I would love to put you in touch with my Director to further define our "ask".
Thanks,
Jose
Sent from my iPhone

On Apr 24, 2017, at 12:37 PM, WaelHamze<EMAIL_ADDRESS>wrote:
I will close this issue for now. Please re-open if you face any issues. Would be good to know if this works for you anyway :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi Jose, thanks for coming back to me. Would be interested in knowing more about your "ask", please send me an email and we will take it from there :)

I've been able to reference your new nuget package in my project solution
without issue and I've also sent you an email from my @rsmus.com email
account.
Thank you Wael!
Jose Quinones
On Mon, Apr 24, 2017 at 3:46 PM, WaelHamze<EMAIL_ADDRESS>wrote:

Hi Jose, thanks for coming back to me. Would be interested in knowing more
about your "ask", please send me an email and we will take it from there :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/WaelHamze/xrm-ci-framework/issues/14#issuecomment-296817343,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALDi1BbNsB3Rqpn4acj6luyjNYsKF-6ks5rzQo8gaJpZM4MbrTT
.


Reorganise the rich build operation details/results types to further decouple from producer side semantics

The goal here is to increase our chances of maintaining backwards compatibility, by making the types that are shared between the core and the scan plugin as light as possible.

I have already asked @eriwen to rename the existing BuildOperationType enum. He agreed and will do that on Monday.

Some insight into the motivation behind each of the added indirections

It's about making it easier to achieve backwards compatibility in two ways

Not hampering the evolution of the producer side of build operations by requiring it to be stable
Minimising the static reach from types that cross the core/scan-plugin boundary in order to minimise accidental leakage of unstable types

now need to write 4-6 types for each build operation

Only for the operations that effectively expose a stable, backwards compatible, API. We are paying a price for that stability in this ceremony. However, I am certain that without this ceremony, we will not be able to keep these types stable.
Counting types is a misleading metric. There's one container type with no behaviour, then two contract interfaces (which are absolutely a requirement), then two implementations. We could do away with the container type and have no way of enforcing compile time assistance for the producer side API, but that seems a might high cost to pay to avoid a few characters.

Only for the operations that effectively expose a stable, backwards compatible, API.

This part convinced me :) As long as the API still allows us to quickly add "simple" operations with no contract towards build scans, we're good.


Use Kotlins Null safety to cleanup code

All the unnecessary "!!" statements that are flagged in the Kotlin TestApp are removed.

remove many of the warnings from #610


Add missing 'functional' tag to test source

Without this tag, gopls under VSCode notes that this file calls undefined functions when no build tags are defined (the default).

@TBBle Thanks for catching this!


eframe_template on mx linux ahs builds desktop but not wasm correctly

I am using egui on mx linux 21 ahs (advanced hardware support, debian 11 bullseye).
When I pull the eframe_template.git and build it for debug or release mode, it works fine. But when I run .build_web.sh and then ./start_server.sh after making minor changes to the app.rs, the changes are never reflected. I attempted the following multiple times, however no updates occurred when changing the /src/app.rs file:

rm -rf ./docs/.
./build_web.sh
./start_server.sh

I had also run ./setup_web.sh earlier on to install wasm_bindgen, etc.
I also made sure to install all the eframe_template dependencies include jq and binaryen. Any help is much appreciate on what I am doing wrong here.
Thanks. James

try disable browser cache.

Thanks for the feedback t18b219k. While the issue doesn't appear to be the browser cache specifically, it does appear to be browser related. I am using Brave browser on mx linux 21 ahs (advanced hardware support) version, and upon toggling the shields for the localhost:8080 site, I eventually got the question to break the application out into a PWA - Progressive Web App. After doing that, it apparently works with updating. Snippet is below. Thanks.

Upon further testing, the toggling of the Brave browser "shields up" button is the main action required to get the updated compiled main.rs to show up - and I don't know why this is. Clearing the cache in the browser has no effect. The Progressive Web App (PWA) from Brave is nice is that is allows the breakout of the page into it's own application window, but that is not the issue either . . . perhaps someone who is very familiar with brave browser will know the function calls when toggling the shields up button . . .


android input issue

Reconsider the input problem of the editor under android, any ideas are welcome.

keep composition input state for Sougou Keyboard #228

Hi @RaulPROP and @BitPhinix,
I have been trying to handle Android input issues for slate-angular. I tested several input cases in the richtext demo, inlines demo, and markdown demo using multiple input methods like Gboard, Sougou Keyboard, Baidu IME, and others. I also validated some issues mentioned in this PR: https://github.com/ianstormtaylor/slate/pull/4988. It has been working fine so far.
Could someone please help me verify whether there are any other problems? Thank you very much!


TypeScript date formatting function bugs

[ ] The DATE Short format is incorrect. Should be MM/DD/YYYY (or whatever is locale appropriate)
[ ] The Regex isn't allowing a space between the format modifier

{{DATE(2017-02-14T06:08:39Z, Short)}}

Expected

Actual

Payload
{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "type": "AdaptiveCard",
  "version": "1.0",
  "body": [
        {
            "type": "TextBlock",
            "text": "Created {{DATE(2017-02-14T06:08:39Z,Short)}} {{TIME(2017-02-14T06:08:39Z)}}",
            "wrap": true
        },
        {
            "type": "TextBlock",
            "text": "Created {{DATE(2017-02-14T06:08:39Z,Long)}} {{TIME(2017-02-14T06:08:39Z)}}",
            "wrap": true
        }
    ]
}

The Date formatter actually supports 3 formats:

Long, e.g. "Monday, February 13th, 2017"
Short, e.g. "Mon, Feb 13th, 2017"
Compact, e.g. "2/13/2017" which you get if no format is specified

Happy to change that, but the above was the original design, so why would the other renderers not match that behavior?
Why should a space be allowed before the format specifier?

Closing in lieu of #908 which will require some changes to all renderers


Ensure Mocked BK client triggers the addEntry callbacks in order

Motivation
There are several unit tests that are failing intermittently due to the fact that the MockBookKeeper (when backported from Bk into pulsar repo) instance is triggering the callback from multiple threads, thus resulting in out-of-order acks that will in turn causes messages to be resent and the tests validations to fail.

run cpp tests

run cpp tests

run cpp tests

rerun integration tests

@merlimat which unit tests were failing because of this?

@merlimat which unit tests were failing because of this?

I was debugging the RawReaderTest in which the assertion was failing because of duplicated messages. It was because of the out-of-order acks that the client was re-sending some of the messages. But I saw the assertion failure at https://github.com/apache/pulsar/blob/48431543729fc4f382637c968976d0d164d5427b/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/OpAddEntry.java#L143 in the logs of several other flaky unit tests in last week.

So is this the root cause of the RawReaderTest failure?

Yes, I was able to reproduce easily when running in a Docker container in MacOS (thus inside a slowing VM)

dunno if it's available for mac bit stress-ng is great for this.
It would be good to have the fixed tests mentioned in the commit message, but I guess it's too late now.


Loading module after building fails

Windows 11
Device: arm64 zygisk
i followed the tutorial for building, after i push the zip file and try to load it in zygisk it says "installation failed" with no other info
if i load it from release version it works, but building it and using that doesnt ??

Hmm that's weird. Maybe try building it with Android Studio. It's the IDE I was using for development. Though just using the command in Powershell is also working fine for me and builds it on Windows 11.
The release version is always built by the CI which is built on linux but either version works for me.
Maybe try with Android Studio first and if it doesn't work maybe upload an example zip that doesn't work to take a look at.

i am building from android studio, i also tried just running the command in powershell
but both didnt work, it might just be my machine's fault, i will try building from wsl later on

i am building from android studio, i also tried just running the command in powershell but both didnt work, it might just be my machine's fault, i will try building from wsl later on

I have same question,do you solve it?


Created a FileDeleteListener that can easily be added in the builder …

Created a FileDeleteListener that detects every deletion from the LRU.
Signed-off-by: Bogdan Stanga<EMAIL_ADDRESS>

@stangabogdan Can you explain how can it be useful? I try to keep code as simple as possible. Each new code should resolve concrete problem. Usually cache hasn't any listener. AndroidVideoCache is just cache. So why it should have this listener?

I think more preferable way is just check exist file or not. See details here

@danikula it's a way to detect whenever a file is deleted. For example, one can keep track of the cached files in a database so he can update the database whenever a file is cached and using this feature, whenever a file is deleted.

@danikula Yes. But what if I want to receive an event when a file is deleted. I cannot check if a file is cached for ever.

@stangabogdan Do you know any image loading library which has same functionality?

@stangabogdan Have to decline PR due to KISS. I don't think this is really needed. Client's code can use method isCached to check cache existence. This call is cheap and quickly. Just use it.


Version 0.10.0 seems to have wrong assembly information

Hi,
I updated my plugin which uses your plugin to the latest version 0.10.0. And it seems that the file and product version does not reflect the actual version:

Thanks, I will get that corrected.


Add nothrow delete operator variant overload.

Fixes the failure case caused by this simple test program:
#include <iostream>
#include <algorithm>
#include <vector>

#include <CppUTest/CommandLineTestRunner.h>
#include <CppUTest/TestHarness.h>

TEST_GROUP(HelloWorld)
{
};

TEST(HelloWorld, OnlyTest)
{
    std::vector<int> v{0, 0, 3, 0, 2, 4, 5, 0, 7};
    std::stable_partition(v.begin(), v.end(), [](int n){return n>0;});
    for (int n : v) {
        std::cout << n << ' ';
    }
    std::cout << '\n';
}

int main(int argc, char** argv)
{
    return CommandLineTestRunner::RunAllTests(argc, argv);
}

Coverage decreased (-0.04%) to 99.841% when pulling 74c56542c105c20f66e4d18eb489a29d6d5316fe on Xiche:nothrow_delete into 11e6b45187853aaa3806440cc01955b144693909 on cpputest:master.

Coverage increased (+0.0001%) to 99.876% when pulling 949b4d65b16f68d698b738d17c9636664cb63c5e on Xiche:nothrow_delete into 11e6b45187853aaa3806440cc01955b144693909 on cpputest:master.

Thanks. I'm a bit worried about compiler support, but I guess we'll see. So far it passes all jobs.


Add TODOs badge to README

Hi there! I wanted to propose adding the following badge to the README to indicate how many // TODO comments are in this codebase:

The badge links to tickgit.com which is a free service that indexes and displays TODO comments in public github repos. It can help surface latent work and be a way for new contributors to find areas of code to improve.
The markdown is:
[![TODOs](https://img.shields.io/endpoint?url=https://todos.tickgit.com/badge?repo=github.com/google/go-github)](https://todos.tickgit.com/browse?repo=github.com/google/go-github)

Thanks for considering, feel free to close this issue if it's not appropriate or you prefer not to add the badge!

Cool! I didn't know about that badge.
Looking through the TODOs, the vast majority are to remove a custom header when the API fully launches.
I'm fine with adding this badge to the top-level README.md.

Given that the vast majority of our TODOs are about removing custom headers, I don't think this actually adds any value to the project, and just adds more noise at the top of our (admittedly, already a little noisy) README file.
(I also don't love the fact that it looks like @patrickdevivo made this kind of drive-by contribution to a dozen different projects on Saturday.  Those often end up being spam contributions, or simply promoting one's own project, though I don't think that's necessarily the case here.)
/shrug

Oh, sorry, @willnorris. We can either revert or we can remove it with another PR... your call.

either way.  And no worries
On Mon, Feb 3, 2020 at 12:03 PM Glenn Lewis<EMAIL_ADDRESS>wrote:

Oh, sorry, @willnorris https://github.com/willnorris. We can either
revert or we can remove it with another PR... your call.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/go-github/issues/1406?email_source=notifications&email_token=AAAAIWBFC7IWX7JXCHO7N4TRBB2BVA5CNFSM4KOXNK6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKVGOFI#issuecomment-581592853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAIWGNQJCYXAAGDNM23FLRBB2BVANCNFSM4KOXNK6A
.

OK, I'll just revert then later tonight.

I’m sorry the issue I opened felt like spam - it certainly was a “drive by”
in an effort to get the word out for a project I’m working on, and I
apologize that didn’t “taste right.” I do promise that the intent of my
opening the issue/PR was genuine in that I do believe the TODO index can
provide value to some projects (as TODO comments are often simply
forgotten).
I understand if you don’t feel like there’s a value add to this repo - it
is funny that pretty much all 161 TODOs are essentially the same phrase!
Thank you for considering it in the first place.
As a side note - I'm hoping to soon make it easy to create Github issues
from TODOs in the index, among other features to make it more of an
"actionable" tool rather than just a big list.
On Mon, Feb 3, 2020 at 3:26 PM Glenn Lewis<EMAIL_ADDRESS>wrote:

OK, I'll just revert then later tonight.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/go-github/issues/1406?email_source=notifications&email_token=AAAN7K37SSVIOGHE44Y2TMLRBB4YNA5CNFSM4KOXNK6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKVIX4Q#issuecomment-581602290,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN7K3T5JZ7TLDZ7RRZMMLRBB4YNANCNFSM4KOXNK6A
.


Description on PyPI isn't rendered properly

A setup.py check -sr shows several issues preventing the description from rendering properly. This gives a somewhat unprofessional first impression, and should be fixed.

Where are those line numbers it spits out pointing at?
➜ tri.declarative git:(master) ./setup.py check -sr
running check
warning: check: missing required meta-data: url

warning: check: Title level inconsistent: (line 418)

warning: check: Title level inconsistent: (line 425)

warning: check: Title level inconsistent: (line 431)

warning: check: Title level inconsistent: (line 440)

warning: check: Title level inconsistent: (line 448)

warning: check: Title level inconsistent: (line 456)

warning: check: Title level inconsistent: (line 465)

warning: check: Title level inconsistent: (line 473)

error: Please correct your package.

Line 418... of what? :(

Of the description in setup.py.

Fixed finally. Phew.


Center traffic lights

Hi
Would be nice to center the traffic lights

Simply put the same number of traffic lights per line as the number of results you get.
Are you living in Belgium ?

Yes I am. weli I just have 3 lights.

So if you put 3 traffic lights per line in the options, this would do the trick.
Bon Week-End

But I do not know it advance the number....

Done in version 5.3. Option (Spread traffic lights)

very cool :) Will test that asap


Have a method in Action to check if vehicle is armable

Essentially, a method to return the same information as bool dronecore::Telemetry::health_all_ok() const

I would say this is sort of fixed by #462.

Only if arming_allowed() / takeoff_allowed are exposed in public API?

That's a good idea actually!

@julianoes I'm full of them. Can you drive it? :-)

Ha! Ok I'll put it on my list.

Implemented: https://github.com/mavlink/MAVSDK/blob/68c8041bc911e31f99c8575a042d6b3919583b39/src/mavsdk/plugins/telemetry/include/plugins/telemetry/telemetry.h#L417.


Use MongoDB database or add support for remote MySQL

Describe the feature you'd like
It would be nice if this would use MongoDB database instead of .sqlite file. It's more stable, and you can get free database from MongoDB atlas.
Additional context
When this is not possible, add support to connect to remote MySQL database instead of using .sqlite file.

Thanks for the suggestion!
that's a pretty good idea! For the current scale of thread-watcher sqlite does the job however if the bot grows sqlite might cause some issues
I'll take a look at what needs to be done

You can use mongoose - https://www.npmjs.com/package/mongoose


Support time type in GenericAvroReader and Writer

I found that GenericAvroReader and Writer do not support the logical type of "time-micros" of Avro.
As a result, it fails when having a transform on a TimeType field, with the following exception:
java.lang.IllegalArgumentException: Unsupported logical type: org.apache.iceberg.shaded.org.apache.avro.LogicalTypes$TimeMicros@6d2abe55
        at org.apache.iceberg.avro.GenericAvroWriter$WriteBuilder.primitive(GenericAvroWriter.java:108)
        at org.apache.iceberg.avro.GenericAvroWriter$WriteBuilder.primitive(GenericAvroWriter.java:49)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visit(AvroSchemaVisitor.java:71)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visit(AvroSchemaVisitor.java:56)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visitWithName(AvroSchemaVisitor.java:85)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visit(AvroSchemaVisitor.java:44)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visitWithName(AvroSchemaVisitor.java:85)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visit(AvroSchemaVisitor.java:44)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visitWithName(AvroSchemaVisitor.java:85)
        at org.apache.iceberg.avro.AvroSchemaVisitor.visit(AvroSchemaVisitor.java:44)
        at org.apache.iceberg.avro.GenericAvroWriter.setSchema(GenericAvroWriter.java:41)
        at org.apache.iceberg.avro.GenericAvroWriter.<init>(GenericAvroWriter.java:35)
        at org.apache.iceberg.avro.AvroFileAppender.newAvroWriter(AvroFileAppender.java:88)
        at org.apache.iceberg.avro.AvroFileAppender.<init>(AvroFileAppender.java:45)
        at org.apache.iceberg.avro.Avro$WriteBuilder.build(Avro.java:163)
        at org.apache.iceberg.ManifestWriter$V1Writer.newAppender(ManifestWriter.java:234)
        at org.apache.iceberg.ManifestWriter.<init>(ManifestWriter.java:72)
        at org.apache.iceberg.ManifestWriter.<init>(ManifestWriter.java:32)
        at org.apache.iceberg.ManifestWriter$V1Writer.<init>(ManifestWriter.java:213)
        at org.apache.iceberg.ManifestFiles.write(ManifestFiles.java:89)
        at org.apache.iceberg.ManifestFiles.write(ManifestFiles.java:74)

@rdblue Would you please reivew this small fix?

Thanks, @waterlx! Could you add a test for this?

@rdblue I updated AvroDataTest to expose the errors in both write and read paths. Could you please review again?

Looks like github didn't pick it up, but Travis shows the tests are passing. I'll merge this. Thanks @waterlx!


Occasional crash with Zombie Awareness when using nether portal

https://paste.dimdev.org/ahupumupef.mccrash - uploaded by VannilaFix

i think this issue is fixed , the original issue has been oppenned 5 years ago and the last update of the original tick dynamic is been uploaded 2 years ago and cannot reproduce it
https://github.com/The-Fireplace-Minecraft-Mods/TickDynamic/issues/27


Consistently use synonyms_of_cp_in_nnn phrase

The naming of multi-word countries in the synonyms_of_cp_in_NNN phrases was
inconsistent. Some used hyphens and one ('south-africa') used an underscore. In
addition, three outcomes were doing :"synonyms_of_cp_in_#{ceremony_country}",
while one was converting hyphens to underscores
("synonyms_of_cp_in_#{ceremony_country}".gsub('-', '_').to_sym).
I've renamed synonyms_of_cp_in_south_africa to
synonyms_of_cp_in_south-africa in the YAML file and removed the special case
of replacing hyphens with underscores in the
outcome_cp_commonwealth_countries outcome.

LGTM :+1:


Fix construction of xcframework architectures

This PR addresses some issues during the construction of the xcframework.
Thin vs fat libraries
When building the xcframework for each platform an own fat library has to be used. But for iphone, simulator, watchos, watchsimulator for each slice a thin library was used. That caused xcodebuild -create-xcframework complains with
'Both x and y represent two equivalent library definitions.'
There was a workaround in gen_xcframework which just skipped the archs iphone64e and watchv7k and so they are also missing in the xcarchive. watchv7kis still required to upload a watchos app to app store.
Missing archs
For WatchOS the arches x86_64 and arm64 are not built. These are required to be able to run a WatchOS 6 or later app in a simulator. So current app watch apps are only able to be developed against a real device.
Dependent changes
Fat library for iPhone moved to dist/iphone
Currently all platforms has a own subfolder in dist/[platform] except for iPhone. It seems to me, that the intention was to have it too, because it was already prepared on same places. I changed that, because otherwise there are always exceptions needed. (e.g. finding the fat platform library for the xcframework)
Fat library for iPhone/Simulator and WatchOs/Watchsimulator separated
Before the fat library dist/lib/libjre_emul.a and dist/lib/watchos/libjre_emul contained architecture for the device and the simulator together. But because of introducing the arm64 simulator arches, they have to be separated. This was the reason why apple introduced the xcframeworks as well.
Together with the change before the new structure is:

dist

lib

iphone

libjre_emul.a

simulator

libjre_emul.a

watchos

libjre_emul.a

watchsimulator

libjre_emul.a

This has to be considered when linking the static libraries.
Renamed watch architecture in J2OBJC_ARCHS
Before watchv7k and watch64 was used to configure the watchos targets via J2OBJC_ARCHS.
These were renamed to watchosv7k and watchos64. Otherwise it's hard to distinguish between watch% wand watchsimulator%.
The official architecture name is watchos anyway. And both notations was  inside the build scripts.
#1713 #1736 #1737

@googlebot I signed it!

These changes look good (thanks!), but they can't be pulled until the CLA issue is resolved. Please read the "What to do if you already signed the CLA" comment above, and see if you can resolve the issue. If not, please email me the details at tball at google.com and I'll see if someone who supports our GitHub account can help.

@googlebot I signed it!

These changes look good (thanks!), but they can't be pulled until the CLA issue is resolved. Please read the "What to do if you already signed the CLA" comment above, and see if you can resolve the issue. If not, please email me the details at tball at google.com and I'll see if someone who supports our GitHub account can help.

Sure. The corporate signing process was still in progress and is successfully finished now.

@tomball There some internal checks failed, but I can't see the issues. Do you have some hints for me? Then I will try to resolve it.

We use Copybara to sync our public repository with our internal one, and the process to sync this pull request with our internal repository kicks off when I approved it last. Normally that works fine, but this time it hit a Copybara bug where the changed files in this PR were copied to new paths instead of updating the files, and those new paths broke the build. I filed a high priority bug with the Copybara team to get this fixed, and will try again as soon as that's done.

Thanks for this contribution, and thanks for your patience while our repository sync tools were repaired!


Home tab title room count pluralization fix

In the tab title, it should read room instead of rooms when there is only one.

Solved in https://github.com/codysherman/2nfm/pull/198


Adds repair capabilities to GPT

This adds two public functions to partition.Table
Verify() which will examine headers and internal data to validate the GPT table to the structure of the disk
Repair() which will update the in-memory table so it can be re-written to disk.

This is super interesting. I am going to make some comments to the code. Do you mind explaining here what the scenario is? Why would we need this, and what this does?

This is super interesting. I am going to make some comments to the code. Do you mind explaining here what the scenario is? Why would we need this, and what this does?

The main scenario here is deploying a partition table on a smaller disk (i.e. 2GB) and then re-writing the contents of that disk (byte-by-byte) to a larger disk (i.e. 4GB). The Linux Kernel, VMware etc.. all complain that the second GPT table is missing and that the first table points to the wrong location.

The main scenario here is deploying a partition table on a smaller disk (i.e. 2GB) and then re-writing the contents of that disk (byte-by-byte) to a larger disk (i.e. 4GB). The Linux Kernel, VMware etc.. all complain that the second GPT table is missing and that the first table points to the wrong location.

I will admit, that is not a scenario I had thought of before. So I create a 2GB disk (actual or image), then dd if=/in/dev of=/out/dev where /out/dev is a disk file or physical disk of size 4GB. When that happens, the first 2GB in out are everything, including the secondary header at the last LBA of the 2GB, following which (from just after 2GB until end of 4GB) is just unused data, perhaps 0s or whatever.
In that case, the kernel/VMware/etc look at the disk size and expect the secondary table at the end, no matter what is in the beginning, and so complain?

Correct 😀

OK, makes sense. Can we address the comments, and then we can run CI and merge in?

@thebsdbox Hi, Thank you very much for your contribution, I wonder if you still have time or interest to complete this PR. It would help us a lot if this feature could be supported, thanks!

Good question.. the development VM I did all this in is long gone, so I'll need some time to remember what I was doing 😂.
I'll add it to my todo list.

@thebsdbox Thanks, your work will help us a lot. And I would like to ask, do you have any update with this PR?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_7c16dd9e-fc3d-438a-9abe-984fc6fb6642/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_7c16dd9e-fc3d-438a-9abe-984fc6fb6642.git

Docs Build status updates of commit f91a826:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-ServiceAccountCannotAccessRepository] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source has enabled Docs Build service account. It seems that at least one of the accounts, e2ebd3, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master#ops-e2e-ppe) and then retry the build. Don't know who to contact? Refer https://repos.opensource.microsoft.com/VSC-Service-Account/repos/docs-archive-test-source/permissions/ to get admin list of the repo if the repo is in one of the organizations Microsoft owns.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Start a guide for contributing third-party extensions

Cut from #93
Paste to new document
Profit

What needs to be added to make this a real thing?

This is a great idea!  A few things come to mind:

Showing actual (pseudo)code of the Source, Sink, and Disposable interfaces.  I'm not sure what the best format for this would be, though.
Providing more guidance on writing a new combinator:

The first choice should always be to build a new combinator using existing ones.  You can do a lot with map, filter, flatMap, loop (or scan), sample, and combine.
If it's not possible, or prohibitive (ie too much capturing required, etc.) to build from existing, then create a new Source+Sink.

Linking to a couple simple, existing combinators as examples, map and filter come to mind because they are dead simple.

Other ideas?

Another:

Showing an example of a producer Source, and maybe linking to an existing one.

Excellent! Don't know of many open source projects that have documented explanations of their internal architecture to help new contributors. I really like the idea. :+1:

@unscriptable Do you want to work more on this?  Or did you want to hand it off?

I'd like to get an initiation version of this merged.  Do you think the right name for this document is "extending", or "architecture"?  Maybe "extending" with an "Architecture" section?

@unscriptable Is it cool with you if I commandeer this and combine it with info in #191?

Ok, new plan: gonna move the info here and in #191 to the wiki.


mytccd edu

Tarrant County College Domain

@Gurumaity I am sorry, the domain my.tccd.edu is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


fix: 🐛 SQFormCheckboxGroup no longer has failed prop error

Changed the initial value of the checkbox group in the story to be an
empty array instead of a string
✅ Closes: #273

Since this only effects storybook let's redo this so it's a chore commit. We don't want to bump the npm version for a code change consumers don't receive.


Autocomplete props

Feature description
Enhance the <Autocomplete /> component adding a couple of useful props:

iconPosition
loading

Icon position
I'm not sure if there was a reason the iconPosition prop isn't available, should the search icon always be on the right for a special reason? Otherwise it would be cool to have this property to decide where to place it. Maybe from a consistency point of view having it always on one side would be best?
Loading status
Having a loading prop to show a spinner or a skeleton would be cool, it can be easily done using a snowflake and the skeleton component but maybe, consistently handling this prop on the DS would be better.
This prop would be used while loading the items of the autocomplete and also when searching in a large database and typing on the autocomplete would load other entries.
Following a video showcasing what I tought could be a solution:
https://user-images.githubusercontent.com/58233289/164181554-d3c8a480-2892-4f8b-8968-9ae63290b82f.mov
Let me know what you think 🙌
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context and information
No response
Code of Conduct

[X] I've searched across open and closed issues for similiar requests

About the first request, textfield props are already available.
https://design.wonderflow.ai/components/inputs/autocomplete#autocomplete

For the second one I need to do some investigation

Ref. https://github.com/wonderflow-bv/wanda/pull/122


Keyboard manager cannot remap keys to get '<' or '>' keys az a result.

Microsoft PowerToys version
0.75.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

step in Settings\keyboard manager\
Remap shortcut

SELECT left side
press RightAlt+CTRL+M
on the right side I want to find the keys '<' or '>' but there is no such an option there

✔ Expected Behavior
There should be '<' and '>' keys in the list of the selection
❌ Actual Behavior
there is no '<' or '>' keys in the list of the selection
Other Software
No response

/dup #19534


search like google

can you improve the search by 👍
now,  search  xxx yyy means xxx or yyy (OR)
can you do google style
key word              meaning
xxx yyy                 xxx + yyy   (and)
'xxx yyy'                exact key word
xxx -yyy                have xxx , but do NOT have yyy
====================
I actually do it in coldfusion in my project.
this is how I do it.

split key word string by space, store into array.
string ( xxx yyy ) --> split --> array[0] = xxx, array[1]= yyy

build sql by
select * from xxx
where
for loop array i
title like array[i]
and
           end for loop

if array[i] first character = '-'
then
array[i]= array[i].remove first '-'
title Not like array[i]

I don't know how your search works, how redis involved.
I think you could do it too.

I found your search code:
controller search.php, I update this to:
$AdvancedSearch = true;
does not work,
How to force use advanced search (AND) for everyone, any user?
$SQLKeywordArray = array();//查询关键字参数数组 $AdvancedSearch = false; $NormalQuery = array();//普通查询条件，用OR连接 $AdvancedQuery = array();//高级查询条件，用AND连接

how to force use AND instead of OR in keyword array?

//关键词预处理
foreach ($KeywordArray as $Key => $KeywordToken) {
//匹配用户名限制条件
preg_match('/user:(.*)/i', $KeywordToken, $SearchUserTopics);
if (!empty($SearchUserTopics[1]) && IsName($SearchUserTopics[1])){
$AdvancedSearch = true;
break;
}
}
I don't understand above code? can you explain?

I think this can close.


Switch from the text DDL type to varchar to avoid type related issues

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

@beikov Why are you proposing to do this?

Remember that we have defined LONG32VARCHAR to mean:

In principle, the type must accommodate strings of length<PHONE_NUMBER>, though this is not an absolutely hard requirement, since such large strings do not occur in practice.

But PostgreSQL looks like it has the following upper limit for varchar:
hreact=# create table V(V varchar(2147483647));
ERROR:  length for type varchar cannot exceed 10485760
LINE 1: create table V(V varchar(2147483647));
                         ^

which is about two orders of magnitude too small.

See the discussion on Zulip for details: https://hibernate.zulipchat.com/#narrow/channel/132094-hibernate-orm-dev/topic/PostgreSQL.20text.20vs.2E.20varchar.20weirdness

For context, blindly casting to text, which is what we are doing right now, leads to issues like this one: https://ci.hibernate.org/view/ORM/job/hibernate-orm-nightly/job/main/687/testReport/junit/org.hibernate.orm.test.function.array/ArrayContainsTest/Build___postgresql_12___Test___testNodeBuilderArray_SessionFactoryScope_/
Performance problems like https://hibernate.atlassian.net/browse/HHH-18780 are also possible.


[CONTENT] Using NFTs in games

This issue is for drafting content for the following:

Title: Using NFTs in games
Address: /tutorial/using-nfts-in-games/

Contributions are welcome! If you'd like to make the global NFT developer community better for everyone by creating a guide for this page, please leave a comment and someone from the core team will be in touch ASAP. 🙏

I'd like to get involved in this. I'm working on the consensys bootcamp and am working on a prototype using NFTs in a game mechanics.

@michaeljohnbennett awesome! would love to get you involved. Do you have something in mind for what could make a good tutorial for beginners?

I'm working on a project for the consensys blockchain bootcamp and I have this NFT game idea I'm developing.
I was thinking it could be a good base for a tutorial. Showing the general logic from minting packs, cards using probabilities.
also some simple on chain logic to do a game. The things I'm working on right now are VRF Oracles (maybe chainlink) but open to ideas on that side.
Sound like something worth working on/with?

Yeah that sounds awesome. As you make progress with your project, let us know and we can support your time to create a tutorial for the basics, if that sounds good! And maybe visit some of the more advanced stuff after?

Yeah I think a fork of the repo would be best to start adding the content! Could save some round trips if you wouldn't mind outlining the sections in this issue that you had in mind for the tutorial so we could align on flow. cc @yusefnapora
Not sure how much code you have written, but if you have something up and working, we could support your work building a tutorial via a microgrant: https://github.com/filecoin-project/devgrants/blob/master/microgrants/microgrants.md cc @mishmosh
Side question - is your project using NFT.Storage? We think it's the easiest tool for the NFT School audience to get hands-on with development, so our goal is to standardize tutorial content using the NFT.Storage libraries.

I haven't seen that yet, I guess its some kind of IPFS storage container? I can use that for sure.
I'll fork the repo this weekend.
I have 3 smart contracts done and tested:

The store contract that allows you to buy packs
The pack NFT which allows you to purchase packs
The cards NFT which you get to play.

I don't have the pack opening random code yet done.

Yup, please check it out - it's a cool product!
Awesome, that sounds like plenty for a tutorial!

I'd like to contribute to this, I'm currently working on two NFT games at the moment.


HelpDot image request for Expensify Classic article for Sage Intacct

Please provide the following images in light mode and in .png format. Whoever completes this task will need a connection to Sage Intacct. I do not have access to this connection and therefore do not know how accurate or feasible a couple of these requests are. Please let me know if any of the steps below are inaccurate.
CONNECT TO SAGE INTACCT
Link to resource: https://help.expensify.com/articles/expensify-classic/connections/sage-intacct/Connect-To-Sage-Intacct
Images needed:

Show how to get to Company > Web Services Users > New.
Show all of the fields completed as follows (all shown in 1 image if possible):

User ID: “xmlgateway_expensify”
Last Name and First Name: “Expensify”
Email Address: Your shared accounting team email
User Type: “Business”
Admin Privileges: “Full”
Status: “Active”

Highlight the subscription link for the user. In another image, select the checkbox next to the Application/Module and highlight it. In another image, highlight the Permissions link.
Show how to get to Company menu > Subscriptions > Time & Expenses. Then in another image, show the toggle switched to subscribe. Then in another image, show the following settings selected (in one image if possible):

Ensure that Expense types is checked.
Under Auto-numbering sequences set the following:
Expense Report: EXP
Employee: EMP
Duplicate Numbers: Select “Do not allow creation”

Continuing with the process from the previous image, click the down arrow on the expense report line and highlight Add. Then in another image, show the following settings:

Sequence ID: EXP
Print Title: EXPENSE REPORT
Starting Number: 1
Next Number: 2

Continuing from the previous steps, highlight the Advanced Settings option. Then in another image, show the following:

Fixed Number Length: 4
Fixed Prefix: EXP

Continuing from the previous step, under Expense Report approval settings, highlight the Enable expense report approval as being unchecked.
Show how to navigate to Time & Expenses and click the plus button next to Employees.
Show how to navigate to Company > Open Setup > Company Setup Checklist > click Import.
Show how to navigate to Time & Expense, Open Setup, and highlight the plus button next to Expense Types.
Show how to navigate to Company > Subscriptions > Customization Services.
In Expensify, go to Settings > Workspaces and highlight the New Workspace button.
Go to Connections > Sage Intacct > and highlight Connect to Sage Intacct. Then in another image, highlight Select Download Package.
In Sage Intacct, show how to navigate to Customization Services > Custom Packages > New Package.
In Sage Intacct, show how to navigate to Platform Services > Custom Packages > New Package.
Show how to navigate to Company > Company Info > Security. Then in another image, scroll down to Web Services Authorizations and show “expensify” (all lower case) added as a Sender ID.
In Expensify, go to Settings > Workspaces > [Workspace Name] > Connections > Configure. Click Connect to Sage Intacct and enter the credentials you’ve set for your web services user. Show these completed fields in the image.

CONFIGURE SAGE INTACCT
Link to resource: https://help.expensify.com/articles/expensify-classic/connections/sage-intacct/Configure-Sage-Intacct
Images needed:

Go to Settings > Workspace > [Workspace name] > Connections and highlight the Configure button next to Sage Intacct.
In Sage Intacct, show how to navigate to Platform Services > Objects > List. Then in another image, Set “filter by application” to “user-defined dimensions” and highlight it.

heya @johncschuster -- assigning you as a liaison for assisting design with getting a few Sage Intacct images for the Exfy Classic. i think you handled the Sage Intacct migration from Community to HelpDot so figured i would assign this one to you to assist. tysm!

Thanks @maddylewis! Yep! I'll work on this with Danny.

Sure thing—and thank you!

Not overdue Melv. We've got a plan!

Notes:
Company Setup Checklist is already complete, or I don't have access to it. I will need to skip that specific screenshot.

Ok! our Sage Intacct instance is set up to use 	User Based permissions, so I followed the flow to connect via User Based permissions and exporting as Expense reports.

@dannymcclain, I kinda went "ham" and have 45 screenshots for you based on the list above.
I'm sure we'll need to sort out which ones are the most valuable, but for now, I'll start by just getting them all to you. What's the best method to transfer all of these?

Awesome! I've just DM'ed you in Slack!

Ok wowee. This is fairly confusing 😅. Might take me a minute to make sense of all these screenshots and the images that have been requested.

Would it be easier to hop on a quick sync! I'm happy to walk through it with you!

I sent over an invite for tomorrow to chat everything over!

Awesome! Here's what I've got so far: https://www.figma.com/design/xDF4dRQIUDzvoc54NOHcnp/Help-Site-Screenshots?node-id=418-8&node-type=canvas&t=3DoB5CcZAaZD4rR7-11
This might be sufficient, or some might be totally wrong haha. If you have time to take a cruise through them before tomorrow it might help us determine if we need to meet/what to talk about when we do. Appreciate your help!

Notes for me on some verbiage change needs:
Standardize on "click the plus button" language in any navigation that calls for it
Step 4.1 Verify the navigation in Sage Intacct matches what is called out in the documentation (no screenshot provided)
Step 4.2 for User Based > Expense reports should be updated to

Go to Company > Time & Expense > Types

OK! Big shout out to John for helping me wrangle and make sense of all these! I've got two zip files of images to share: one is for the Connect article and one is for the Configure article.
All the images have file names that correspond to what step of the article they're showing/related to.

ExpensifyHelp_Sage_Connect.zip
ExpensifyHelp_Sage_Configure.zip

Ok Melvin I feel like this is not overdue. John and I Saged and conquered ☝

I just need to upload these. I'll get them done today!

Ok! My article edits were rolled back so the images could be uploaded in their own PR. I'll get the edits for the articles back up shortly.

Sorry for the delay on this! My text edits were all rolled back when the images were uploaded.
I will get the alt text updated today

Finally got the docs finished finished.

Ah! Neil has a few formatting fixes for me in the PR. I'll work on those this afternoon.

I couldn't get to the formatting fixes yesterday. I will slot them in this afternoon.

Ok! I've taken Neil's suggestion on how to fix the formatting. We'll see if that resolves it!

Ugh, I misunderstood what Neil needed. He needed me to indent one image, and I indented all of the images. I will fix that today.

Ok! I fixed the mistaken indented images and asked for a re-review.

Bumped the PR for a re-review

Waiting on the re-review

It's merged! Now we're waiting on the deploy 🙌

Deployed to staging! 🎉

Waiting on the deploy to prod

No payment is required for this one. It was reviewed by internal engineers.


Update repos.gradle

Gradle doesn't like http anymore

Overriding and merging for gradle 7 upgrade.


Add more checks for the galera MaaS plugins

This commit will add some more robust checks for the galera
plugin. We also create some alarms for the corresponding checks.
The criteria for these alarms include(when the alarms will trigger):

Open_files > open file limit
Innodb_row_lock_time_avg > Innodb_row_lock_time_max
Innodb_deadlocks != 0
Aborted_clients > 1/s[1]
Aborted_connects > 1/s[1]
Access_denied_errors != 0

[1] 1/s ish. The rate function modifier is used here.
By definition:
Rate as defined by this formula where V=value, and T=time:
(V1 - V0) / (T1 - T0).
This is more of a ratio as to how many clients/connects are
being aborted every {{ maas_check_period }} seconds (default is 60)
Connects #1164

TODO items:
Add alarm for aborted clients and aborted connects.
Do research on what the threshold for innodb_row_lock_time_avg should be.

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

recheck_swift

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

recheck_swift

RPC-AIO-Liberty-PR: Fail

tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops[compute,id-7fff3fb3-91d8-4fd0-bd7d-0204f1f180ba,network,smoke]
recheck_swift

One additional note. we're missing one check that was requested:
Max_used_connections in relation to max_connection setting
Currently we check current number of connections vs max connection setting, but not max_used_connections.

blocked until I can figure out why https://github.com/rcbops/rpc-openstack/pull/1242/files#diff-50fff0dae21040cc684fefc936d5f01bR42 keep putting that alarm in the CRITICAL state

Turns out it was just a typo on my end.
I've removed the status-blocked/dont-merge and re-added the status-needs-review labels.

 Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.
19:17:22     <class 'neutronclient.common.exceptions.PortNotFoundClient'>
recheck_swift
recheck_ceph

RPC-AIO-Liberty-PR: Fail

tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops[compute,id-7fff3fb3-91d8-4fd0-bd7d-0204f1f180ba,network,smoke]
recheck_swift

RPC-AIO-Liberty-PR: Fail

tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops[compute,id-7fff3fb3-91d8-4fd0-bd7d-0204f1f180ba,network,smoke]
recheck_swift

Waiting for @BjoernT's feedback. Other than that, this PR is ready for review

In regards to max_used_connections we just want to get an alarm when it is close to max_connections. We don't have to report necessarily percentage metrics, I would probably prefer numbers so we can see how the numbers changed once we increase max_connections over a history.
In regards to queries metric, yes we should fix that and convert it into a queries per second.
Reporting overall number make no sense IMHO. Just highlight it in the release notes

In regards to queries metric, yes we should fix that and convert it into a queries per second.
Reporting overall number make no sense IMHO. Just highlight it in the release notes

We can't actually report a per-second thing with the plugins. We don't get that metric from Galera and the plugin is run at the intervals that the monitoring agent is configured to run it. That can, however, be created in the intelligence portal. Further, support isn't the only consumer of the metrics, so I would rather have a deprecation cycle for the metric as @claco described.

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

I've opened a separate bug for the queries_per_second issue so we don't mix things up and for tracking purposes.
https://github.com/rcbops/rpc-openstack/issues/1254
PR for this fix to come..

@alextricity25 Do we need to triage/prioritize 1254? How severe is it and does it block you completing #1242?

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

@b3rnard0 #1254 Won't block this one, but it does need to be prioritized and triaged.

If you alarm on a mysql status variable that is essentially a high tide mark, that alarm will never clear.
Better to alarm on current connection average over a number of polls (this can be expressed in the alarm language).

If we can separate alarm from metric we can leave the alarm at percentage, the actual metric is preferred to be reported as counter rather than percentage

If we can separate alarm from metric we can leave the alarm at percentage, the actual metric is preferred to be reported as counter rather than percentage

I believe that's what was being proposed this entire time.

I've updated the PR  so the alarm for max_seen_connections compares against a percentage threshold, using max_connections to derive the percentage [1]
The max_seen_connections(maps to the max_used_connections mysql status variable) metric will stay, so it can still be monitored via the RAX intelligence dashboard as requested by @BjoernT.
@sigmavirus24 @andymcc @BjoernT Could you guys take another look to review when you get the chance? Thanks!
[1] https://github.com/rcbops/rpc-openstack/pull/1242/files#diff-50fff0dae21040cc684fefc936d5f01bR42

I'm still not convinced that using a high tide mark to alert on is a good thing. Once that goes in to alert, it will not clear without manual intervention (by either restarting mariadb, or logging in and manually resetting that status variable in mysql).  I would argue that you would be best using the alarm language to calculate the average current connections over the last n polls, and alert when that average breaches a percentage of max_connections.

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

I agree with @mancdaz - @BjoernT is that your intention for support to have to take manual action for the max_seen_connections alert every time, since once it alerts it will never clear until a flush status or some other manual action is taken (e.g. service restart).
This seems like a poor alert mechanic - it seems more sensible for it to clear automatically if it has been seen and dealt with, and would really generate an alert in the same way as the max_connections vs current_connections (except it would never clear) - since the max_connections vs current_connections alarm is already setup I'm not sure what this would add.
Saying that - if support deem this to be important I don't have an issue approving the PR.

This is not an issue for us, if max_used_connections hits to close to max_connections we have to do something anyway. Either rising the limit or flush the status if it was a problem. I'm ok with the alarm being critical in that case

Gate looks to be failing because the access denied_errors is already above 0 after build, so it starts off in error - maas-verify fails.
Unless we want to clear the status of all those vars before setting up MaaS (which we should definitely not do at all), we'll need a new approach to that (e.g. setting it a lot higher, which also seems like a very bad approach).
@BjoernT what issues are we aiming to detect by seeing the access_denied_errors? Surely this implies some service is unable to access mysql correctly - so we should be seeing issues with the service rather than relying on mysql access_denied errors.

Access denied should always be 0 if not then this is a hint that a service is misconfigured.
I would not say that we can expect a service error in this case, this check would just add another layer of monitoring to monitor the database rather than just the service side

@BjoernT the essence of the problem is that the counter doesn't start at 0, so it is already in alert - failing the gate (and alerting straight away).
Additionally flush status won't reset that - in both cases support will need to make changes that aren't related to the error. E.g. error causes max connections/denied errors to generate - support resolves the issue, but alert remains. Ticket doesn't clear, and if nothing further is done then either no further alerts will happen (if a subsequent issue occurs), or it will continue to create alerts.
I'm not sure I understand what this gives additional to what is already setup (max_connection check will do the exact same except it will clear when the issue is resolved, and access_denied is a flat counter that could fire as a result of pretty much any misconfigured client connection - plus it lets you know of a client side server issue, which seems to be best found on the client itself).

Why doesn't it start as 0 it does for me, it should be 0. Do we know why it is not 0 in gating ?

@BjoernT somebody trying to access mysql from a client and maybe mistyping a password is going to increase this counter. Surely this is not a sensible thing to be alerting on?
I think anywhere we're trying to alert on high tide marks or cumulative counters is simply doing it wrong.

Yes that's why I asked to monitor rates not counter so we know if counter is growing. If denied counter is already at >0 then we can take it off if we are not able to monitor rates.

A rate would be ok - @alextricity25 can you change that to be a rate of increase of the access_denied_errors value.
I think we should just remove the max_connections_seen alarm though.

Made new commit:

Added release note
changed access_denied_errors alarm criteria to use the rate function modifier
added thresholds for the access_denied_errors alarm in defaults/main.yml
removed the mysql_seen_connections alarm

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

recheck_all

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

18:18:00 Captured traceback:
18:18:00 ~~~~~~~~~~~~~~~~~~~
18:18:00     Traceback (most recent call last):
18:18:00       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/scenario/manager.py", line 108, in delete_wrapper
18:18:00         delete_thing(*args, **kwargs)
18:18:00       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/services/compute/floating_ips_client.py", line 62, in delete_floating_ip
18:18:00         resp, body = self.delete(url)
18:18:00       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/common/rest_client.py", line 290, in delete
18:18:00         return self.request('DELETE', url, extra_headers, headers, body)
18:18:00       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/services/compute/base_compute_client.py", line 53, in request
18:18:00         method, url, extra_headers, headers, body)
18:18:00       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/common/rest_client.py", line 645, in request
18:18:00         resp, resp_body)
18:18:00       File "/opt/tempest_c2c90574a78c7704f06e8fac279c4f1d11365993/tempest/lib/common/rest_client.py", line 764, in _error_checker
18:18:00         message=message)
18:18:00     tempest.lib.exceptions.ServerFault: Got server fault
18:18:00     Details: Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.
18:18:00     <class 'neutronclient.common.exceptions.PortNotFoundClient'>
recheck_ceph
recheck_swift

RPC-AIO-Ceph-Liberty-PR: Fail

19:26:43 TASK: [Verify Checks & Alarms are registered] *********************************
19:45:18 failed: [jrpcaioiad-123] => {"attempts": 8, "changed": false, "cmd": ". /openstack/venvs/maas-UNKNOWN_RELEASE/bin/activate\n /opt/rpc-openstack/scripts/rpc-maas-tool.py verify-created --entity jrpcaioiad-123", "delta": "0:00:44.090162", "end": "2016-07-25 19:45:17.061452", "failed": true, "failed_when_result": true, "rc": 1, "start": "2016-07-25 19:44:32.971290", "stdout_lines": ["--- 2016-07-25 19:45:17.047960 ---", "The following checks have config files but are not registered with the maas api: ['lb_api_check_nova--unspecified_lb'] ", "The following alarms are present in config files but are not registered with the maas api: ['lb_api_alarm_nova'] "], "warnings": []}
19:45:18 stdout: --- 2016-07-25 19:45:17.047960 ---
19:45:18 The following checks have config files but are not registered with the maas api: ['lb_api_check_nova--unspecified_lb']
19:45:18 The following alarms are present in config files but are not registered with the maas api: ['lb_api_alarm_nova']
19:45:18 msg: Task failed as maximum retries was encountered
19:45:18
19:45:18 FATAL: all hosts have already failed -- aborting

recheck_ceph

I know this has gone on for a bit - but 2 more changes! Hopefully the last 2.

Updated commit:

Added threshold variables for aborted_connects and aborted_clients
Changed the open_files alarm criteria to check against a percentage and a defined threshold.

👍  LGTM - assuming the gate passes.
Thanks for making those changes!

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

18:13:12 TASK: [Verify Checks & Alarms are registered] *********************************
18:30:42 failed: [jrpcaioiad-740] => {"attempts": 8, "changed": false, "cmd": ". /openstack/venvs/maas-UNKNOWN_RELEASE/bin/activate\n /opt/rpc-openstack/scripts/rpc-maas-tool.py verify-created --entity jrpcaioiad-740", "delta": "0:01:04.209792", "end": "2016-07-26 18:30:41.391258", "failed": true, "failed_when_result": true, "rc": 1, "start": "2016-07-26 18:29:37.181466", "stdout_lines": ["--- 2016-07-26 18:30:41.374708 ---", "The following alarms are present in config files but are not registered with the maas api: ['aborted_connects--jrpcaioiad-740-galera-container-7e4b42f3'] "], "warnings": []}
18:30:42 stdout: --- 2016-07-26 18:30:41.374708 ---
18:30:42 The following alarms are present in config files but are not registered with the maas api: ['aborted_connects--jrpcaioiad-740-galera-container-7e4b42f3']
18:30:42 msg: Task failed as maximum retries was encountered
18:30:42
18:30:42 FATAL: all hosts have already failed -- aborting
recheck_all
Also going to clean up some of the Apsu bot comments

RPC-AIO-Ceph-Liberty-PR: Fail

RPC-AIO-Liberty-PR: Fail

RPC-AIO-Ceph-Liberty-PR: Fail

recheck_ceph

RPC-AIO-Ceph-Liberty-PR: Fail

tempest.scenario.test_server_basic_ops.TestServerBasicOps.test_server_basic_ops[compute,id-7fff3fb3-91d8-4fd0-bd7d-0204f1f180ba,network,smoke]
recheck_ceph

👍 passing gate now.

👎 based on comments inline

Removed labels because the connected issue is used to track these pieces of information.

UPDATE: The latest commit makes the changes @mancdaz pointed out.

This PR is ready for another look @mancdaz, @andymcc, @BjoernT.

The code looks good and no errors that I saw during AIO build. 👍

👍

liberty12-2 backport here: https://github.com/rcbops/rpc-openstack/pull/1269


Forbid update on LastVotedForkSlots and HeaviestFork and log conflicting messages.

Do not allow anyone to update LastVotedForkSlots and HeaviestFork (except for updating observed_stake).
This simplifies our code, and we don't see a reason anyone should keep changing it.

why do we need to update observed_stake

Because we exit when 80% see 80%, otherwise you may exit too early when
your HeaviestFork haven’t propagated
On Tue, Aug 27, 2024 at 20:46 carllin @.***> wrote:

why do we need to update observed_stake
—
Reply to this email directly, view it on GitHub
https://github.com/anza-xyz/agave/pull/2731#issuecomment-2314146656, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FJ5FMYHH2RVB4QMVEYZQLZTVB2XAVCNFSM6AAAAABNCDXEFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJUGE2DMNRVGY
.
You are receiving this because you were assigned.Message ID:
@.***>

Ah I see, this isn't the stake of the validator voting, but the observed stake from that validator for the heaviest fork, which of course can change as that validator detects more stake voting. Makes sense!


Further skinning

Brought in patternfly, added openshift logo and changed the header text a little

As a suggestion - could we structure the header (the top black bar) so that it is a single line of
"OpenShift Origin > Logs for project cookies "
The logo is interesting, but I think ends up wasting space that would be better used for conveying a single goal.  Logs for project cookies is the effective page H1.  Origin is in theory part of that H1, because it's identifying that this entire thing is part of one concept.  Is there an alternative patternfly logo behavior?

Yup, I could do that...  I followed the same guidelines as was outlined for the Java console, be fairly easy to inline those two lines.


How to enable LOG_LEVEL=debug ?

Hello,
my backstage application is running on the EKS cluster, and I would like to enable LOG_LEVEL=debug, can you please provide me instructions to enable that?

@nvadluri do you have an example of your Backstage k8s deployment YAML file? The short answer is you need to set the LOG_LEVEL environment variable for you deployment however you set environment variables
You can set LOG_LEVEL either in a k8s configmap which your deployment uses to populate it's environment from or you can add the environment variable directly to the container for the deployment.
Check out https://humanitec.com/blog/handling-environment-variables-with-kubernetes if you are unsure how to do this

thank you for the reply, and I enabled it by adding the env variable directly to my deployment, and it's working now...thank you so much for the response

@jrusso1020 thank you for the reply, and I enabled it by adding the env variable directly to my deployment as you mentioned, and it's working now...thank you so much for the help. (I have another issue, if you can please check that once and provide any suggestions, would be very helpful)


fix(storage): make sstable iter valid while fetching block

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Currently, in next of sstable iterator, we will first advance the block iterator, and if the block iterator it invalid, we will then fetch the next block in an await. However, when the future is dropped, the sstable iter will be at an invalid state, and in the upper level merge iterator, we wrap the sstable iter with a Heap::PeekMut, which when dropped, will get the key of sstable iter to do binary comparison, panic because the sstable iter is at an invalid state.
In this PR, we fix this issue by introducing try_next for block iterator, which will not advance when it is going to be invalid and return false, notifying the sstable iter to fetch next block. In this way, while the sstable iterator is fetching the next block, the block iterator is still valid, and there will not be any panic.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)

Related issue: https://github.com/risingwavelabs/risingwave/issues/6313


[📚] Codelab link for Firebase Dynamic Links returns a 404 not found error

The Codelab for the firebase dynamic links package (https://codelabs.developers.google.com/codelabs/flutter-firebase/#4) returns a 404 Not Found error when visited.
If the Codelab has been removed then perhaps it should be deleted from the readme.

I've removed the link from the readme as this codelabe no longer exists.


Telegram API Terms of Service

The task is to implement a mobile application using the tdlib library. The application will interact with our third party server and with the Telegram server. Authorization in Telegram and access to chats and groups will be implemented through the tdlib library. As planned, the application will send (directly or through a telegram bot) and store on our third party server data about visited chats and groups (chat ID, group ID, etc.).
Question: What information and how can we send and store on our third-party server without violating the Telegram security policy? If it's allowed.

https://core.telegram.org/api/terms#1-privacy--security:
1.1. Telegram is a privacy-oriented platform. All client apps must, therefore, guard their users' privacy with utmost care....```


Batch

Are you not supposed to add it in constants.py as wel'l ?

'code/batch' already exists in constants.py


feat: enhance sizeBasedFlushStrategy by judging WAL disk usage

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
This PR links this issue,  #842 WAL disk usage based flush strategy

If WAL takes up too much disk space, the recovery process may take a long time to finish. So, to fix the issue, I enhance SizeBasedFlushStrategy by judging WAL disk usage. If the disk usage exceeds the threshold, the data in memtable should be flushed.

Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)
#842

Looks like some CI workflow fails.

Sounds reasonable!

Hello, Any update? @ShenJunkun

Hello, Any update? @ShenJunkun
Sorry, I am currently busy with my undergraduate graduation project, and there is no update yet. I will update this issue in my free time recently.

Hello, Any update? @ShenJunkun
Sorry, I am currently busy with my undergraduate graduation project, and there is no update yet. I will update this issue in my free time recently.

Good luck.No rush, just a reminder.

Hello, Any update? @ShenJunkun
Sorry, I am currently busy with my undergraduate graduation project, and there is no update yet. I will update this issue in my free time recently.

Good luck.No rush, just a reminder.

Thx!

I'd like to close this PR since we already implement a new flush strategy mentioned in #1462


Executable name conflicts with borg backup utility

https://github.com/borgbackup/borg also uses the "borg" command, and is already present in several linux distributions including Arch, Debian, and Ubuntu. Given its popularity, I might suggest modifying the executable name somewhat to avoid conflict with it.

This is a completely valid point, as an Ubuntu user (on servers) I never encountered borg, but I realise this may be an issue.
I will think about a name change or an other solution, meanwhile I suggest just changing the exec name if you already have the other borg installed - it won't affect anything, since the borg client is just a tiny self contained library.


Average run time.

Could you please tell me how long does it take to color a photo? 7-8 minute per photo for me.

@AliAsafPolat I tested on the image with a resolution of 256x256 pixels on a notebook GTX 1070, the first step to compute bidirectional correspondences takes about 18 seconds while the similarity network and colorization networks take around 0.2 seconds.

I just tested on the image with resolution of 256x256 and reference image with 256x256, it takes 2 minutes 40 seconds. My notebook has GTX 950M. It changes a lot again. Is not it?

As i checked in task manager, similarity subnet works on CPU colorization subnet works on GPU. How can i fix that problem?


Provide sprite version of the emoji images

We all :heart: sprites. Sprites are fast. :stars:
I don't want to have to generate my own sprite for emojis when I build my JS app. I don't want to have to host my own sprite on my possibly-non-CDNed site.
Would it be out of scope of the project to provide a sprite (and accompanying stylesheet) for the emoji icons?

we tested and sprite was everything but fast. Much heavier to download, doubled in problems/maintenance requiring CSS too. Sprites also didn't work as image src.

How about a JSON or similar file mapping emoji to the image graphic data URI?

Yes, I misunderstood why he suggested sprites and have learned a number of things since I made that comment. Thanks.

we tested and sprite was everything but fast. Much heavier to download, doubled in problems/maintenance requiring CSS too. Sprites also didn't work as image src.

Hello @paulshapiro :) I am just curious, so why Twitter is using sprites on emoji picker if the sprites are slow? see: https://abs.twimg.com/responsive-web/client-web/twemoji_sprite.0fcc5555.png 😉👍


Bug: contributor attributed twice for same file

You can see this in action in https://github.com/open-sauced/landing-page/pull/362/files. Look for @zeucapua and you'll notice it appears twice on multiple files, e.g.
components/sections/navigation/Header.tsx @brandonroberts @zeucapua @zeucapua

This doesn't break anything and it's still a valid CODEOWNERS file, but it should be cleaned up.

Which version was this on? I think this was fixed recently so my guess is this was from an older version of the CLI

Which version was this on? I think this was fixed recently so my guess is this was from an older version of the CLI

Looks like it was v1.5.0-beta.2.0.20240913205418-ea2b4a9594e9

Confirmed that with 2.1.0, this is still an issue.
Get latest from the open-sauced/landing-page project and run pizza generate codeowners ./. Notice Zeu appears twice.

Confirmed that with 2.1.0, this is still an issue.

Huh weird - I'll look into this. It shouldn't be able to have double the attributions since it uses a straight mapping of usernames to emails.

Noticed the same:
pizza version
Version: v2.1.0
Sha: a659d86cd3a88d1856f4f1bd345e2b4d92fa4dd5
Built at: 2024-09-16-22:55:27

.eslintrc.js @takanome-dev @takanome-dev
.github/workflows/development.yml @renovate[bot]
.prettierrc @takanome-dev @takanome-dev
CHANGELOG.md @El Hadji Malick Seck @El Hadji Malick Seck @dependabot[bot]
env.mjs @takanome-dev @takanome-dev
log.log @takanome-dev @takanome-dev
next.config.mjs @El Hadji Malick Seck @El Hadji Malick Seck
package.json @El Hadji Malick Seck @El Hadji Malick Seck @renovate[bot] @takanome-dev @takanome-dev

As a side note: this won't affect GitHub's CODEOWNERS automation. It'll still be able to automatically request review from whomever is attributed on that file

You can see this in action in this latest PR https://github.com/open-sauced/landing-page/pull/383. It's using v.2.2.0 of the pizza CLI, https://github.com/open-sauced/landing-page/actions/runs/11197287184/job/31127055020


Angular Base Project doesn't work. Module build failed: main.ts is not part of the compilation.

Versions
Angular CLI: 1.6.0
Node: 8.8.1
OS: win32 x64
Angular: 5.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
@ angular/cli: 1.6.0
@ angular-devkit/build-optimizer: 0.0.35
@ angular-devkit/core: 0.0.22
@ angular-devkit/schematics: 0.0.41
@ ngtools/json-schema: 1.1.0
@ ngtools/webpack: 1.9.0
@ schematics/angular: 0.1.10
@ schematics/schematics: 0.0.10
typescript: 2.4.2
webpack: 3.10.0
Repro steps

Install Angular if not done previously with: npm install -g @angular/cli
Create a new app: ng new my-app
Go to the app folder and launch: ng serve --open
When the browser opens and tries to access the page, it will show a "ERR_CONNECTION_CLOSED" message, and an error will be shown in the console where the "ng serve" command was launched.

Observed behavior
Cannot launch a freshly created Angular Project (Unmodified). When launched with "ng serve --open" this error is shown. (The same error is shown when "ng build" is used)
 ng serve --open
** NG Live Development Server is listening on localhost:4200, open your browser on http://localhost:4200/ **
Date: 2017-12-07T13:11:24.434Z
Hash: f5ec8ec9adb18d697f1f
Time: 3634ms
chunk {inline} inline.bundle.js (inline) 5.79 kB [entry] [rendered]
chunk {main} main.bundle.js (main) 3.51 kB [initial] [rendered]
chunk {polyfills} polyfills.bundle.js (polyfills) 1.22 kB [initial] [rendered]
chunk {styles} styles.bundle.js (styles) 9.07 kB [initial] [rendered]
chunk {vendor} vendor.bundle.js (vendor) 854 kB [initial] [rendered]

ERROR in D:/Users/bor/Desktop/Project/my-app/src/main.ts
Module build failed: Error: D:\Users\bor\Desktop\Project\my-app\src\main.ts is not part of the compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
    at AngularCompilerPlugin.getCompiledFile (D:\Users\bor\Desktop\Project\my-app\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:643:23)
    at plugin.done.then (D:\Users\bor\Desktop\Project\my-app\node_modules\@ngtools\webpack\src\loader.js:467:39)
    at <anonymous>
 @ multi webpack-dev-server/client?http://<IP_ADDRESS>:0 ./src/main.ts
ERROR in D:/Users/bor/Desktop/Project/my-app/src/polyfills.ts
Module build failed: Error: D:\Users\bor\Desktop\Project\my-app\src\polyfills.ts is not part of the compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
    at AngularCompilerPlugin.getCompiledFile (D:\Users\bor\Desktop\Project\my-app\node_modules\@ngtools\webpack\src\angular_compiler_plugin.js:643:23)
    at plugin.done.then (D:\Users\bor\Desktop\Project\my-app\node_modules\@ngtools\webpack\src\loader.js:467:39)
    at <anonymous>
 @ multi ./src/polyfills.ts

webpack: Failed to compile.

Desired behavior
It should show the basic template demo.
Mention any other details that might be useful (optional)

I was not able to reproduce with a project I just created... could you please post the contents of these files:

[root]/tsconfig.json
[root]/src/tsconfig.app.json

Reproduced on Windows 10, using both Git Bash from git version 2.14.1.windows.1 and Windows Command prompt. I did not check PowerShell.
Node versions in the background were both 8.40 x64, 8.9.3 x64 LTS, 9.2.1 x64. TypeScript is always at 2.5.3.
Tested on Arch Linux with NodeJS 9.2.1 x64 and TypeScript 2.5.3 succeeded on the other hand.
This seems to be an OS-dependent issue to me.
tsconfig.json:
{
  "compileOnSave": false,
  "compilerOptions": {
    "outDir": "./dist/out-tsc",
    "sourceMap": true,
    "declaration": false,
    "moduleResolution": "node",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "target": "es5",
    "typeRoots": [
      "node_modules/@types"
    ],
    "lib": [
      "es2017",
      "dom"
    ]
  }
}

src/tsconfig.app.json
{
  "extends": "../tsconfig.json",
  "compilerOptions": {
    "outDir": "../out-tsc/app",
    "baseUrl": "./",
    "module": "es2015",
    "types": []
  },
  "exclude": [
    "test.ts",
    "**/*.spec.ts"
  ]
}

On my side completely uninstalling nodejs and reinstalling the latest version of nodejs solved the issue for me. I never removed nom, and it took my old nom version an it's dependencies. I still don't know what that error means, but it is gone for me for now and everything compiles properly now. Not sure if I should close the issue now or not. Seems to be after all some nodejs conflict or something like that.

Npm* instead of nom.

At least on Windows, uninstalling NodeJS and then re-installing it also uninstalls / installs npm, as they are bundled into a single installer.
I've done my tests using update installations; this did not solve the issue for me. Also, running a full uninstallation / installation cycle of Angular CLI at global level did not change anything.

I'm sorry, but we can't reproduce the problem following the instructions you provided.
Remember that we have a large number of issues to resolve, and have only a limited amount of time to reproduce your issue.
Short, explicit instructions make it much more likely we'll be able to reproduce the problem so we can fix it.
If the problem persists, please open a new issue following our submission guidelines.
A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.


Allow only requesting the oauth scopes that an app needs

Is your feature request related to a problem you're having? Please describe.
My app needs to write new files to Google Drive, and it needs nothing else. As such, the only scope I request is https://www.googleapis.com/auth/drive.file. However, the login screen suggests that the app will be much more invasive than it is:

It appears that additional scopes beyond what the consuming app requests are always requested:
https://github.com/google/GoogleSignIn-iOS/blob/main/GoogleSignIn/Sources/GIDSignInInternalOptions.m#L56
https://github.com/google/GoogleSignIn-iOS/blob/main/GoogleSignIn/Sources/GIDScopes.m#L58
Specifically it looks like the email and profile scopes are always added.
My assumption is that those additional scopes are unnecessary for basic usage, but please tell me if I am incorrect.
As an additional downside, requesting extraneous scopes makes implementing Google's new granular permissions more messy and complicated.
Describe the solution you'd like
In the spirit of the principle of least privilege, it should be possible to use Google SignIn without extraneous scopes added. To do this in a backwards compatible way, the current behavior should remain the default, but it should be possible to set a flag to avoid scopes being added.
This seems like a very small change.
Describe alternatives you've considered
First, it's possible that those other scopes are necessary for all Google API usage. However, the fact that granular permissions allows rejecting them, that seems false.
Second, it's possible that consumers of Google SignIn just live with requesting extra scopes. However, that makes their apps look more invasive than they need, and thus sketchy. Further, it means that developers need to unnecessarily handle the more complicated granular permission case where you request many scopes.
Additional context
I'm happy to contribute this feature if it would be accepted.

Ah, it looks like this worked as we would like in 5.0.0 and switched to the more invasive behavior by 5.0.2, which is why we didn't notice this issue in older versions. So, this is functionally a user-experience regression.

I cannot speak to why this behavior changed between v5.0.0 and v5.0.2. My guess is that it had something to do with the idea that GSI is primarily focused on providing features associated with signing in, and providing, a user. The profile scope comprises basic information for the currently signed in user. I suspect the email was considered integral as well.
To be honest, I do not think it is likely that we change the current automatic scope requests for profile and email. It is definitely not going to be something we do in the next release (v7.2.0).
I can leave this issue open for a little bit to see if others are interested in a change. I could imagine an API where developers can modify the automatic scopes: i) the default where email and profile are requested, and ii) the modification where developers ask for some subset of those.

Gotcha, thanks so much for your thoughts! That makes sense, but do you know if there is a suggested alternative route for developers who are only using APIs like Drive or Sheets? My understanding is that Google recommends this tool for that use case, so it seems good to either support that use case well or for Google to recommend a different tool.


[BUG]: CSI Powermax chooses ISCSI protocol over NVMeTCP

Bug Description
CSI Powermax chooses ISCSI protocol over NVMeTCP
Logs
time="2024-07-22T11:30:11Z" level=info msg="valid NVMeTCP initiators: [nqn.2014-08.org.nvmexpress:uuid:xx_xxxx]"
time="2024-07-22T11:30:11Z" level=info msg="valid (existing) iSCSI initiators (must be manually created): []"
time="2024-07-22T11:30:11Z" level=info msg="iscsid.service is not active. Current state - inactive"
time="2024-07-22T11:30:11Z" level=info msg="Attempting to start iscsid.service"
time="2024-07-22T11:30:11Z" level=info msg="Successfully started ISCSI daemon"
time="2024-07-22T11:30:11Z" level=info msg="setting up array 000xx00012xx for Iscsi, host name: csi-node-NVM-master-1-khpvceulyzhg2 masking view ID: csi-mv-NVM-master-1-khpvceulyzhg2"
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
Install driver with node having both iSCSI and NVMeTCP initiators
Set Transport protocol to Auto=""
Expected Behavior
Choose NVMeTCP over iSCSI
CSM Driver(s)
CSI Powermax
Installation Type
Operator
Container Storage Modules Enabled
No response
Container Orchestrator
K8s 1.30
Operating System
RH

link: 26442